Skip to content

Commit 99c77be

Browse files
committed
docs(session-management): clarify required local checks (format + build) and mandate quick manual CLI test before review
1 parent 654025b commit 99c77be

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

docs/session_management_plan.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,10 @@ import plugin, { sessionClearDefaultsLogic } from '../session_clear_defaults.ts'
354354
- Always run locally before requesting review:
355355
- `npm run typecheck`
356356
- `npm run lint`
357+
- `npm run format:check`
358+
- `npm run build`
357359
- `npm run test`
360+
- Perform a quick manual CLI check (mcpli or reloaderoo) per the Manual Testing section
358361

359362
### Minimal Changes Policy for Tests (Enforced)
360363

@@ -374,7 +377,7 @@ import plugin, { sessionClearDefaultsLogic } from '../session_clear_defaults.ts'
374377

375378
At the end of each numbered step above:
376379

377-
1. Ensure all checks pass: `typecheck`, `lint`, `test`.
380+
1. Ensure all checks pass: `typecheck`, `lint`, `format:check`, `build`, `test`; then perform a quick manual CLI test (mcpli or reloaderoo) per the Manual Testing section.
378381
- Verify tool descriptions comply with the Tool Description Policy (concise, no session-defaults mention).
379382
2. Stage only the files for that step.
380383
3. Prepare a concise commit message focused on the “why”.

0 commit comments

Comments
 (0)