chore(format): make eslint and prettier agree, then clear the debt in one pass

[xenodeve]

PRD: #275 · Blocks the "make lint required" issue.

EN

lint cannot become a check while three things are true at once:

• nestjs's lint script is eslint --fix — it rewrites the working tree, so running it is a mutation, not a check. It is why 484 files sit permanently modified and why every commit needs manual path-intersection to avoid staging unrelated churn.
• bunx prettier --check fails on 204 files at HEAD, so the tree has never been prettier-formatted.
• eslint --fix and the standalone prettier config format differently, so the two disagree and running either dirties the other's work.

Acceptance criteria

• One formatter of record is chosen and the two configurations are made to agree — verified by running both and observing no diff.
• lint (check, no writes) is split from lint:fix in both workspaces, and the root lint uses the checking form.
• One formatting-only PR clears the debt across the repository. It contains no behavioural change, stated in its body so review can be a spot check rather than a line-by-line read.
• The pre-existing no-base-to-string errors in missing-column.ts and the pg-*.store.ts files are fixed as part of this, not left as a reason to keep the check off.
• After the pass, bun run lint at the root exits zero on a clean checkout and leaves git status unchanged — asserted, because "it does not write" is the whole point.
• bun test green in both workspaces afterwards; a formatting pass must not change behaviour.

Not in scope: making lint a required check (its own issue, immediately after this one).

TH

PRD: #275 · บล็อกอิชชู "ทำให้ lint เป็น required"

lint กลายเป็น check ไม่ได้ตราบใดที่สามข้อนี้จริงพร้อมกัน:

• script lint ของ nestjs คือ eslint --fix — มัน เขียนทับ working tree ฉะนั้นการรันมันคือการแก้ไข ไม่ใช่การตรวจ · เป็นเหตุที่ไฟล์ 484 ไฟล์ค้างสถานะ modified ถาวร และเป็นเหตุที่ทุก commit ต้องตัดกับ path ด้วยมือเพื่อเลี่ยงการ stage ของที่ไม่เกี่ยว
• bunx prettier --check ล้ม 204 ไฟล์ที่ HEAD ฉะนั้น tree ไม่เคยถูกฟอร์แมตด้วย prettier เลย
• eslint --fix กับ prettier config ที่แยกอยู่ ฟอร์แมตต่างกัน ฉะนั้นทั้งสองขัดกันและการรันอันใดก็ทำให้งานของอีกอันเปื้อน

เกณฑ์การยอมรับ

• เลือกตัวฟอร์แมตที่ถือเป็นทางการหนึ่งตัว และทำให้สองคอนฟิกตรงกัน — ยืนยันด้วยการรันทั้งคู่แล้วไม่เกิด diff
• แยก lint (check ไม่เขียน) ออกจาก lint:fix ในทั้งสอง workspace และ lint ที่ root ใช้รูปแบบที่ตรวจอย่างเดียว
• PR ฟอร์แมตล้วนหนึ่งอันล้างหนี้ทั้งรีโป · มัน ไม่มี การเปลี่ยนพฤติกรรมใด ระบุไว้ในเนื้อ PR เพื่อให้การรีวิวเป็นการสุ่มตรวจ ไม่ใช่การอ่านทีละบรรทัด
• error no-base-to-string ที่มีมาก่อนใน missing-column.ts และไฟล์ pg-*.store.ts ถูกแก้เป็นส่วนหนึ่งของงานนี้ ไม่ใช่ถูกทิ้งไว้เป็นเหตุผลให้ปิด check ต่อ
• หลังผ่านรอบนี้ bun run lint ที่ root คืนศูนย์บน checkout ที่สะอาดและทำให้ git status ไม่เปลี่ยน — assert ไว้ เพราะ "มันไม่เขียนไฟล์" คือประเด็นทั้งหมด
• bun test ผ่านทั้งสอง workspace หลังจากนั้น · การจัดฟอร์แมตต้องไม่เปลี่ยนพฤติกรรม

ไม่อยู่ในขอบเขต: การทำให้ lint เป็น required check (อิชชูของตัวเอง ถัดจากอันนี้ทันที)

[xenodeve]

EN — nestjs delivered + merged; the nextjs formatter decision stays open

The nestjs half of #280 is done: lint is now a check (no --fix), eslint and prettier agree on one root formatter, the formatting debt is cleared (PR #298, merged fb85ecc), and bun run lint at the root exits 0 on a clean checkout with git status unchanged.

Still open — the nextjs decision (yours): nextjs/ is not prettier-formatted. Its lint is eslint-config-next (no prettier) and passes; pulling it into a prettier pass would reformat ~177 files and change its quote style (double → single). Decide: (a) adopt prettier for nextjs too (one full frontend pass), (b) keep nextjs under eslint-config-next only (add nextjs/ to the format scripts' scope explicitly), or (c) close this and file it separately. Until decided, nextjs/ is excluded from the pass via .prettierignore.

TH — ครึ่ง nestjs เสร็จและ merge แล้ว; การตัดสินใจ formatter ของ nextjs ยังเปิด

ครึ่ง nestjs ของ #280 เสร็จ: lint ตอนนี้เป็นการตรวจ (ไม่มี --fix) · eslint กับ prettier เห็นพ้องบน root formatter เดียว · ล้างหนี้ฟอร์แมตแล้ว (PR #298, merge fb85ecc) · และ bun run lint ที่ root ออก 0 บน checkout ที่สะอาดโดย git status ไม่เปลี่ยน

ยังเปิด — การตัดสินใจ nextjs (ของคุณ): nextjs/ ยังไม่ได้ถูก prettier จัดฟอร์แมต · lint ของมันคือ eslint-config-next (ไม่มี prettier) และผ่านอยู่ · การดึงมันเข้า prettier pass จะจัดฟอร์แมต ~177 ไฟล์และเปลี่ยน quote style (double → single) · ตัดสินใจ: (a) รับ prettier สำหรับ nextjs ด้วย (pass ทั้ง frontend), (b) ให้ nextjs อยู่ใต้ eslint-config-next อย่างเดียว (เพิ่ม nextjs/ ในขอบเขต format scripts อย่างชัดเจน), หรือ (c) ปิดอันนี้แล้วยื่นแยก · จนกว่าจะตัดสินใจ nextjs/ ถูกกีดกันจาก pass ผ่าน .prettierignore