Skip to content

[Maintain 0007] Pre-v3.0.0 maintenance#675

Merged
waleedkadous merged 2 commits intomainfrom
builder/task-NH8C
Apr 17, 2026
Merged

[Maintain 0007] Pre-v3.0.0 maintenance#675
waleedkadous merged 2 commits intomainfrom
builder/task-NH8C

Conversation

@waleedkadous
Copy link
Copy Markdown
Contributor

Summary

First MAINTAIN run on the simplified 2-phase protocol. Pre-release hygiene for v3.0.0.

  • Dead deps removed: http-proxy + @types/http-proxy (flagged in run 0006, now acted on).
  • New devDep: @xterm/xterm pinned explicitly instead of relying on transitive resolution through codev-dashboard (per 3-way review).
  • Docs sync: arch.md directory trees refreshed (afx commands, bin scripts, codev/protocols/, spir/ subtree, codev-skeleton/protocols/); version stamp bumped to v3.0.0-rc.9.
  • Doc sync: AGENTS.md pulled back in line with CLAUDE.md — local-install flow for pnpm monorepo (drift since PR Migrate monorepo from npm to pnpm #669).
  • Lessons: 3 entries from Spec 653 (builder handling / TICK removal) added to Critical section in lessons-learned.md.
  • Run file: codev/maintain/0007.md.

Run details: codev/maintain/0007.md has the full audit/clean/sync/verify log, deferred items, and consultation log.

Verification

  • pnpm build → green
  • pnpm test (codev package) → 2462 passed, 13 skipped, 122 files
  • diff CLAUDE.md AGENTS.md → empty

3-way review

Consulted Gemini, Codex, and Claude on commit 96fbea1 before applying their feedback in commit 2d4a17f:

Reviewer Verdict Outcome
Gemini REQUEST_CHANGES All addressed (protocol-schema.json, skeleton tree, Critical section placement, @xterm/xterm pin)
Codex APPROVE_WITH_COMMENTS spir/ subtree fix + skeleton tree addressed; stale CLI examples elsewhere deferred
Claude APPROVE_WITH_COMMENTS Verified all claims in run report against filesystem

Test plan

  • CI green on the branch
  • Architect: confirm arch.md trees match actual layout
  • Architect: confirm lessons-learned.md additions are generalizable

- Remove unused http-proxy dependencies (flagged in run 0006, now acted on)
- Refresh arch.md directory trees (afx commands, bin scripts, protocols)
- Bump arch.md version stamp to v3.0.0-rc.9
- Sync AGENTS.md with CLAUDE.md (local-install flow for pnpm monorepo)
- Add 3 lessons from Spec 653 review to lessons-learned.md
- Add maintenance run file codev/maintain/0007.md
- arch.md: add protocol-schema.json to codev/protocols/ tree
- arch.md: correct spir/ subtree (remove nonexistent manifest.yaml, add
  protocol.json, builder-prompt.md, prompts/, consult-types/)
- arch.md: refresh codev-skeleton/protocols/ subtree (add aspir, air,
  bugfix, spike)
- lessons-learned.md: move Spec 653 lessons into Critical section
- packages/codev: add @xterm/xterm as explicit devDependency instead of
  relying on transitive resolution through codev-dashboard
- codev/maintain/0007.md: document consultation findings and actions
@waleedkadous waleedkadous merged commit 7f7cfe4 into main Apr 17, 2026
6 checks passed
@waleedkadous waleedkadous deleted the builder/task-NH8C branch April 17, 2026 22:15
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