docs(status): refresh STATUS through the module substrate era - #83
Conversation
The doc was last updated 2026-07-12, when the engine was built but not yet live and the UI/deploy were still ahead. Bring it current with everything shipped since: - Live deployment (HT-43): engine at desk.resonantiq.app, Agent Inbox UI at inbox.resonantiq.app, on Supabase + a real Gmail mailbox; live-only Gmail threading bugs found and fixed (HT-49/50). - Agent Inbox UI live (HT-23) and per-Agent identity/login/auth (HT-51, HT-54). - Inbound observability (HT-44, migration 19). - Module substrate era: open-core catalog (HT-66) and substrate v1 spec (HT-67) under specs/modules/; implementation waves 1-3 (HT-68/69/70, migrations 020-023 — actor model, events/webhooks, assistant auth + drafts + approval); first module scaffolded born-proprietary (HT-73). The HT-71 operator guide (docs/modules/, PR #81) is still open, so it is listed under Next rather than Done. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
📝 WalkthroughWalkthroughSTATUS.md is refreshed to describe Helpthread’s live mail deployment, closed-loop threading, Agent Inbox API contract, module substrate v1, operational surfaces, updated roadmap, and deferred scope. ChangesHelpthread status documentation
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@STATUS.md`:
- Around line 9-10: Update the AI actor terminology in STATUS.md to use
“Assistants” consistently, including the prose references near the changed
“assistant” text and the additional indicated occurrences. Preserve “Agents” for
human support staff and leave any literal repository or path identifiers
unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
| substrate v1** is built and merged: typed events, signed webhook delivery, and assistant (AI) | ||
| actors that authenticate and post draft-before-send work for an Agent to approve — the extension |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Use “Assistants” consistently for AI actors.
These changed prose references use lowercase assistant for AI actors. Apply the required vocabulary consistently, while preserving literal repository and path names if they are identifiers.
Proposed wording
-assistant (AI) actors
+Assistants (AI actors)
-draft-reply assistant
+draft-reply AssistantAs per coding guidelines, human support staff must be referred to as “Agents” and AI actors as “Assistants”; never conflate them in schemas, code, documentation, or prose.
Also applies to: 22-25, 72-72, 76-76
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@STATUS.md` around lines 9 - 10, Update the AI actor terminology in STATUS.md
to use “Assistants” consistently, including the prose references near the
changed “assistant” text and the additional indicated occurrences. Preserve
“Agents” for human support staff and leave any literal repository or path
identifiers unchanged.
Source: Coding guidelines
…sics (#91) PR #83 landed already-stale: main moved by nine PRs while it was in review, and it shipped one claim that contradicts the charter. - HT-71 operator guide was listed as an open PR under Next. #81 merged at 17:48, ~2h before #83 merged at 19:39, and docs/modules/ has been on main since. Moved to Done with its precision follow-up (#84). - Marketplace was listed under 'Not yet / deferred'. CHARTER §3/§4/§5 were amended the same day (HT-79, #86) to make it a launch-day component of Phase 3 — built now, proven as the dogfood install path. Removed from deferred; marketplace v1 spec (#87, draft) now leads Next. - Added inbox basics (HT-76/77/78, #90, migrations 24-25): saved replies & macros, snooze, send & close — shipped engine features with no STATUS line. - Added catalog reclassification (HT-75, #82): KB and end-user portal are paid, 71-module gap audit closed, open-core line restated. Passkeys stay core, reconciled in #85. - Added passkey login spec (HT-75, specs/auth/passkeys.md, draft.3) to Next. Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
STATUS.md was last updated 2026-07-12, when the engine was built but not yet live and the UI/deployment were still ahead. This brings it current with everything shipped since — every claim verified against the repo (git log, merged PRs,
specs/modules/,src/db/migrate.ts), not memory.What changed
The top-line now leads with the live, dogfooded reality (engine at desk.resonantiq.app, Agent Inbox UI at inbox.resonantiq.app, Supabase + real Gmail mailbox), and the era since 07-12 is captured under Done:
specs/modules/; implementation waves 1–3 (HT-68/HT-69/HT-70, PRs feat(engine): module substrate wave 1 — actor model, drafts, assistants, webhooks, event outbox (HT-68) #77/feat(engine): module substrate wave 2 — events, outbox drain, webhook delivery, admin API (HT-69) #79/feat(engine): module substrate wave 3 — assistant auth, drafts API, draft-approval (HT-70) #80, migrations 020–023 — actor model, events/webhooks, assistant auth + drafts + approval); first module scaffolded born-proprietary inHelpthread/module-draft-assistant(HT-73).One accuracy correction worth flagging
The substrate operator guide (
docs/modules/, HT-71) is PR #81 — still open, not merged, anddocs/modules/does not yet exist onmain. It is therefore listed under Next, not Done. (Migrations 020–023 all landed together in wave 1 / PR #77, not spread across the three waves.)🤖 Generated with Claude Code
Summary by CodeRabbit