Skip to content

[#1004] docs: Remove stale AgentChattr claims from repository guidance and README credits#1012

Merged
realproject7 merged 1 commit into
mainfrom
task/1004-remove-agentchattr-claims
Jul 17, 2026
Merged

[#1004] docs: Remove stale AgentChattr claims from repository guidance and README credits#1012
realproject7 merged 1 commit into
mainfrom
task/1004-remove-agentchattr-claims

Conversation

@realproject7

Copy link
Copy Markdown
Owner

Closes #1004

Summary

Docs-only. Removes references portraying AgentChattr / agentchattr-telegram as current QuadWork dependencies, and describes the verified current arrangement (built-in file-based JSONL chat + MCP shim). Scope limited to the three files named in #1004.

Changes

  • README.md (External Tools credit block): removed the AgentChattr credit bullet. Per the ticket, no credit should imply a current dependency; the remaining credits (gh, Claude Code, Codex, Next.js/Express, node-pty, xterm.js) are untouched.
  • CLAUDE.md (What is QuadWork): AgentChattr chatthe file-based project chat; stale docs/PROPOSAL.md pointer (file does not exist) → README.md for the full product overview.
  • CLAUDE.md (Dependencies): replaced the two stale bullets (AgentChattr external dependency; external realproject7/agentchattr-telegram bridge) with the current, verified arrangement — built-in file-based JSONL chat (server/file-chat.js) exposed via MCP shim (server/mcp-chat-shim.js) as chat_send/chat_read, and the built-in optional Telegram bridge (server/bridges/telegram.js).

EPIC Alignment

  • Batch 24 ticket docs: Remove stale AgentChattr claims from repository guidance and README credits #1004 owns only README External Tools credits, AGENTS.md, and CLAUDE.md stale-AgentChattr guidance. This diff touches exactly README.md + CLAUDE.md; AGENTS.md needed no change (see below).
  • Shared invariant (describe current file-based JSONL chat + MCP shim, current arrangement, only when verified): every replacement cites a real current source file confirmed to exist.
  • Out of scope, untouched: README narrative owned by docs(readme): Reframe QuadWork around governed batch delivery #1003, runtime code, tests, package versions, legacy cleanup/migration implementation (bin/quadwork.js cleanup --legacy, server/migrate-ac.js, server/ac-restore.js), and realproject7/quadwork-web.

Self-Verification

  • Acceptance — rg -i agentchattr README.md AGENTS.md CLAUDE.md → no matches (exit 1).
  • AGENTS.md: the top-level file is untracked (locally seeded from templates/seeds/dev.AGENTS.md); it and all tracked templates/seeds/*.AGENTS.md contain zero AgentChattr text, so the acceptance criterion is already satisfied there — no edit made (minimal-change).
  • Current-code verification of replacement claims:
    • File-based JSONL chat: server/file-chat.js:23 (general.jsonl), :169 (fs.appendFileSync, mode 0o600).
    • MCP shim tools: server/mcp-chat-shim.js:27 (chat_send), :39 (chat_read), :135 (serverInfo name "quadwork-chat").
    • Built-in Telegram bridge: server/bridges/telegram.js (present; discord.js alongside).
    • docs/PROPOSAL.md confirmed absent (ls → No such file); repointed to README.md (present).
    • Telegram bridge is now built-in, not the external agentchattr-telegram repo; remaining agentchattr strings live only in legacy cleanup/migration paths (bin/quadwork.js) which are explicitly out of scope and untouched.
  • Scope: git diff --statCLAUDE.md (+3/-3), README.md (-3); no other files staged.
  • CI: pending on push.

Acceptance criteria

  • rg over exactly README.md, AGENTS.md, CLAUDE.md finds no AgentChattr/agentchattr text
  • No credit implies a current dependency
  • Current file-chat + MCP-shim arrangement described only where verified
  • Legacy cleanup/migration implementation and tests untouched
  • quadwork-web not edited

…README credits

Remove references portraying AgentChattr / agentchattr-telegram as current
QuadWork dependencies and describe the verified current arrangement.

- README.md: drop the AgentChattr External Tools credit (no credit implies a
  current dependency).
- CLAUDE.md: replace "AgentChattr chat" + stale docs/PROPOSAL.md pointer with
  the file-based project chat and README overview; replace the AgentChattr /
  external agentchattr-telegram dependency bullets with the built-in
  file-based JSONL chat + MCP shim (server/file-chat.js, server/mcp-chat-shim.js,
  chat_send / chat_read) and the built-in optional Telegram bridge
  (server/bridges/telegram.js).

Docs-only; no runtime code, tests, package versions, legacy cleanup/migration
implementation, or quadwork-web touched. AGENTS.md already contains no
AgentChattr text (rg over the three files finds none).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@project7-interns project7-interns left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verdict: APPROVE

Epic Alignment: PASS

The two-file docs diff removes the stale dependency portrayal required by #1004 and preserves the #1005 boundary.

Checked (evidence)

  • Ticket contract: live gh api repos/realproject7/quadwork/issues/1004 → only README External Tools, AGENTS.md, and CLAUDE.md; diff changes README.md:254-260 and CLAUDE.md:2-8, 47-52 only.
  • Architecture claims: current head server/file-chat.js:20-25,169 establishes JSONL storage and 0o600 writes; server/mcp-chat-shim.js:27-39,135 defines chat_send/chat_read and quadwork-chat.
  • Acceptance scan: current-head README/CLAUDE plus local AGENTS.md → rg -i agentchattr README.md AGENTS.md CLAUDE.md has no matches; tracked tree has no top-level AGENTS.md.
  • Riskiest part of this diff: replacing external-service claims; the referenced built-in Telegram bridge exists at server/bridges/telegram.js, and the replacement accurately marks it optional.
  • Kill-list: scanned all new diff ranges — clean.
  • CI: gh pr checks 1012test pass (50s).

Findings

  • None.

Decision

The diff is minimal, its public architecture claims are verified against the current head, and it meets every #1004 acceptance criterion.

@realproject7

Copy link
Copy Markdown
Owner Author

@re2 Independent Review — APPROVE ✅ (head cc5d3ea)

Checked (evidence)

  • Acceptance criterion (zero AgentChattr text in the three files) — verified on PR head: git show cc5d3ea:README.md | grep -ic agentchattr0; same for CLAUDE.md0. AGENTS.md is untracked (seeded); all tracked seeds templates/seeds/{dev,head,re1,re2}.AGENTS.md0 matches, so the generated AGENTS.md is clean without an edit. ✓
  • New CLAUDE.md dependency claims are realserver/file-chat.js, server/mcp-chat-shim.js, server/bridges/telegram.js all exist on head; shim exposes chat_send (server/mcp-chat-shim.js:27) and chat_read (:39). ✓
  • Stale pointer fixdocs/PROPOSAL.md is genuinely absent from the tree (git cat-file -e cc5d3ea:docs/PROPOSAL.md → absent); CLAUDE.md correctly redirects to README.md. ✓
  • Scope — diff touches only CLAUDE.md (+3/−3) and README.md (−3, External Tools credit bullet). No runtime/tests/legacy-migration/quadwork-web changes. ✓
  • CIgh pr checks 1012 → test pass (50s). ✓

Non-blocking note: templates/seeds/butler.CLAUDE.md still points at the absent docs/PROPOSAL.md, but that file is outside #1004's scope and correctly untouched.

Verdict: APPROVE on cc5d3ea. (Recorded as a comment; shared bot account cannot self-approve on GitHub.)

@realproject7
realproject7 merged commit 5f16c0a into main Jul 17, 2026
1 check passed
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.

docs: Remove stale AgentChattr claims from repository guidance and README credits

2 participants