Skip to content

docs: codify agent-workflow discipline learned in production - #33

Merged
fengzhi09 merged 1 commit into
mainfrom
docs/agent-workflow-discipline
Sep 26, 2026
Merged

fengzhi09 merged 1 commit into
mainfrom
docs/agent-workflow-discipline

Conversation

@fengzhi09

Copy link
Copy Markdown
Collaborator

What

Adds a "Discipline learned the hard way" block to AGENTS.md's feedback-driven workflow section, codifying the rules that emerged from this development cycle:

  • Self-check before acceptance — dev subagents live-verify render paths first (a crash-on-render bug once passed 8 unit tests); tests must drive internal code paths, not just exported names (an export-alias bug shipped dead code green).
  • Process safety — PID-only signaling; pattern kills hit unrelated instances (including the live dev server — twice).
  • Test hygiene — MCODE_WEBUI_* tmp overrides for server-spawning tests (lint-enforced in test:release-tools) and teardown handle closure (leaked handles hung CI twice).
  • Model routes — current dev/acceptance route matrix and the mid-task route-swap procedure.

Verification

node scripts/source-inventory.mjs — pass (4574 files; content-only edit, no inventory change). Full pnpm verify deferred to CI.

Record the hard-won rules in the feedback-driven workflow section: live
self-check of render paths before acceptance, PID-only process signaling
(pattern kills hit unrelated instances), test isolation env vars and
teardown handle hygiene (enforced by the test:release-tools lint), and the
current subagent model-route matrix with mid-task route-swap procedure.
@fengzhi09
fengzhi09 merged commit ebf15f7 into main Sep 26, 2026
8 checks passed
@fengzhi09
fengzhi09 deleted the docs/agent-workflow-discipline branch September 26, 2026 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant