{"content":"Dev journal: workspace-instructions tests wired into CI + npm scripts (review round 2)\n\nCommit 69bb9afc on fix/workspace-instructions-tests — closes the second review flag (\"orphan tests\"): workspaceInstructionsBoundary.test.mjs and coworkSessionDetailGitBranch.test.mjs were only covered by the ad-hoc `node --test tests/*.test.mjs` glob, which neither CI nor any npm script runs.\n\nChanges:\n1. CI (.github/workflows/build.yml): new \"Run workspace-instructions contract tests\" step — `node --test tests/workspaceInstructionsBoundary.test.mjs tests/coworkSessionDetailGitBranch.test.mjs`. Both are pure file-read + regex source-contract tests (no compile step, no platform deps), safe on every matrix OS; runs right after the runtime-dependency-contract step.\n2. npm script: `test:workspace-instructions` added to package.json so the pair can be run locally in one command (and is discoverable next to the other grouped suites).\n3. Review suggestion: folder selector hint now mentions symlinked files in both EN and ZH i18n copy; component comment and the boundary test assertions updated to match.\n\nVerified: npm run test:workspace-instructions 6/6 PASS, i18n coverage 3/3 PASS, eslint clean, renderer tsc 0 errors. Lesson recorded: any future static source-contract test must land in build.yml (or a named npm script) in the same commit, or it does not count as wired.","contentType":"text/plain;utf-8","attachments":[],"quotePin":""}