docs: milestone sync — the "what Orphus authors" maps missed an arc of first-party code - #104
docs: milestone sync — the "what Orphus authors" maps missed an arc of first-party code#104kelvincushman wants to merge 2 commits into
Conversation
The four-stage harness arc landed as one PR, and the maps that tell a reader
which half of this tree is vendored did not move with it. Both said Orphus
authors roundtable and fleet — so a contributor reading them would conclude the
browser extension and the capability boundary are upstream's, and either report
a bug to Atomic or leave the files alone.
- AGENTS.md gained @orphus/transcribe, which was a workspace package no agent
map listed. Its entry says the quiet part: not bundled, fails closed, and that
is the intended state rather than a guard to remove.
- README's tree gained packages/transcribe and stops calling coding-agent
wholly Atomic-derived.
- architecture.md's path table gained transcribe and the four coding-agent
subsystems, with the real paths (core/{capabilities,replay}, provider-audit.ts,
cli/inspect-runtime.ts, extensions/browser, core/terminal/termdom-*).
- docs/README.md's two-halves note matches, and no longer sends "a question
about the harness underneath" upstream — harness.md is about Orphus's own
capability boundary, which is the opposite of what that sentence implied.
No changelog entry: documentation-only, no shipped behaviour changes.
Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WSJaWPmbk52kSjoJi9TQ1a
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 23 minutes Limit details: You’ve used the included review currently available. Your 64 included PR review attempts over the past 7 days set your current allowance at 1 review per hour. Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab. How can I continue?Wait for the limit to reset, then comment An organization admin can change what happens after included review limits in Billing. How do review limits work?CodeRabbit enforces per-developer PR review limits within each organization. For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
Comment |
The milestone docs sync AGENTS.md asks for after a feature arc lands, run against
mainatab5070d(#101 merged).What was actually wrong
Two files tell a reader which half of this tree is vendored and which half Orphus wrote. Both still said roundtable and fleet. After #101 that is wrong in a way that changes behaviour: a contributor reading
docs/architecture.mdwould conclude the browser extension and the capability boundary are Atomic's, and either report a bug upstream or leave the files alone.AGENTS.md@orphus/*packages;@orphus/transcribewas not one of themREADME.mdpackages/transcribe/;coding-agentlabelled simply "(Atomic-derived)"docs/architecture.mdcoding-agentsubsystems, by real pathdocs/README.mdTwo findings beyond the obvious
@orphus/transcribewas a workspace package no agent-facing map listed. Its entry now says the quiet part explicitly — not bundled, fails closed, and that is the intended state, not a guard to remove. Without it, the next agent to notice the package is unregistered has every reason to "fix" it."A question about the harness underneath is usually answered upstream." That sentence in
docs/README.mdpredatesharness.md, which documents Orphus's own capability boundary and session records. The word now points at both halves at once, and the sentence sent readers to exactly the wrong one. Rewritten, with the ambiguity called out rather than left for someone to trip over.Verification
Every path named in the new table was checked to exist before it was written — the first draft said
packages/coding-agent/src/harness/, which does not exist; the real locations arecore/{capabilities,replay}/,core/provider-audit.ts,cli/inspect-runtime.ts,extensions/browser/, andcore/terminal/termdom-*. Thetest/unit/{harness,browser,terminal,transcribe}-*glob matches 16 files.No changelog entry: documentation-only, no shipped behaviour changed.
Generated by Claude Code