Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion codev-skeleton/.claude/skills/update-arch-docs/SKILL.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
---
name: update-arch-docs
description: Audit, prune, and update the project's architecture documentation — `codev/resources/arch.md` and `codev/resources/lessons-learned.md`. Use this skill when running MAINTAIN's arch-doc step, or when asked to update / audit / prune `codev/resources/arch.md` or `codev/resources/lessons-learned.md`. The skill is opinionated about what NOT to include in these files (per-spec changelogs, exhaustive enumerations, aspirational state, duplicate meta-spec content) and ships two operating modes: diff-mode (apply a specific change) and audit-mode (propose cuts with reasons). It edits files directly via normal file-edit tooling and does not invoke destructive shell commands.
description: "Audit, prune, and update the project's architecture documentation — `codev/resources/arch.md` and `codev/resources/lessons-learned.md`. Use this skill when running MAINTAIN's arch-doc step, or when asked to update / audit / prune `codev/resources/arch.md` or `codev/resources/lessons-learned.md`. The skill is opinionated about what NOT to include in these files (per-spec changelogs, exhaustive enumerations, aspirational state, duplicate meta-spec content) and ships two operating modes: diff-mode (apply a specific change) and audit-mode (propose cuts with reasons). It edits files directly via normal file-edit tooling and does not invoke destructive shell commands."
---

# update-arch-docs
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,17 @@
id: bugfix-922
title: bugfix-update-arch-docs-skill-
protocol: bugfix
phase: verified
plan_phases: []
current_plan_phase: null
gates:
pr:
status: approved
requested_at: '2026-05-29T03:14:44.610Z'
approved_at: '2026-05-29T03:15:31.989Z'
iteration: 1
build_complete: false
history: []
started_at: '2026-05-29T02:53:38.613Z'
updated_at: '2026-05-29T03:15:39.037Z'
pr_ready_for_human: false
32 changes: 32 additions & 0 deletions codev/state/bugfix-922_thread.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# bugfix-922 thread

## Investigate
Reproduced the YAML parse error with the snippet in the issue. Root cause is
exactly as described: unquoted `description:` scalar containing
`two operating modes: diff-mode` — the bare `: ` makes the parser try to
nest a mapping inside the scalar.

## Fix
One-line change: wrapped the entire `description:` value in double quotes.
No embedded double quotes in the string, so no escaping needed. Verified
with the issue's repro snippet (now exits 0, parses successfully, prints
the description starting with "Audit, prune, and update...").

Scope adhered to: only `codev-skeleton/.claude/skills/update-arch-docs/SKILL.md`
modified. The three live working copies (`.claude/`, `.codex/`, `.gemini/`)
are intentionally left untouched per issue scope.

## Flaky tests
`pnpm test` fails locally on 3 pre-existing flaky tests in
`packages/codev/src/terminal/__tests__/session-manager.test.ts`, tracked in
#905. Not fixed here — out of scope.

Note on intermediate history: I initially skipped these to unblock porch's
phase-advance check (per the role file's "Handling Flaky Tests" section),
but the architect corrected me — #905 documents that the same skip was
already attempted and reverted from PR #904 as out-of-scope. The skip
commit was reverted (`fe08391c`); diff vs main for the test file is empty.

## PR
Pushing to `mohidmakhdoomi/codev` fork per builder preference; PR
fork → upstream (`cluesmith/codev`).
Loading