Repository navigation
copy(faq): drop the unsourced one-loop premise from FAQ 6 - #2890
Conversation
The answer opened "For a single agent, we agree", agreeing with a premise no file on main supports (the #2884 audit scores it FALSE). It now opens on a neutral line and keeps the walked claims. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MDs9DQ4A4V1Vdp54isbxo5
|
No actionable comments were generated in the recent review.
|
| Layer / File(s) | Summary |
|---|---|
Update agent loop wording frontend/src/content/landing-faq.json |
The answer describes loop design as a choice within each agent. It retains the descriptions of team work, task claims, and connector grants. |
Priority:
Estimated code review effort: 1 (Trivial) | ~3 minutes
Change: Other
Possibly related PRs
- Team-Commonly/commonly#2886: Updates FAQ 6 wording about shared conversations and removes its subagent statement.
- Team-Commonly/commonly#2884: Audits the landing FAQ and identifies the FAQ 6 premise addressed here.
Merge Risk: 
Minimal · up to 41a99
-
The FAQ update is limited to clarifying loop design within an agent and is ready to merge.
Pre-merge checks: 
4
Finishing Touches
-
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Comment @coderabbitai help to get the list of available commands.
lilyshen0722
left a comment
There was a problem hiding this comment.
D138 copy gate at 41a9902. CI is green; every check completed as success or skipped. There are no review threads.
- The opener answers the question and claims nothing. The question at
landing-faq.json:23asks "Should one agent run as one loop instead of subagents?". "That's a choice inside each agent." answers it by saying how each agent runs is that agent's own choice. It makes no claim about what Commonly does. That is accurate: the Claude Code adapter's subagent cue (claude.js:113-116) is a runtime choice, not something the platform imposes. The question's second half, "Isn't Commonly a multi-agent system?", is still answered by the unchanged "Across a team…" sentence. - No claim is lost. The removed text is "For a single agent, we agree: each expert should be one strong loop." That is the premise my #2884 audit scored FALSE: "we agree" agreed with an unsourced claim (Landing FAQ 6 (premise, in the question)). Changing "several of those experts" to "several agents" only drops the back-reference to that sentence. "Divides by expertise and ownership" and "each running under its own owner" are still there, so the TRUE row is unchanged. A word diff confirms the rest of the answer is byte-identical.
- There is a single source.
landingFaq.ts:1importslanding-faq.jsonintoLANDING_FAQ, andbuildFaqPageJsonLd(:30-37) emits each answer verbatim.generate-seo-pages.mjs:589reads the same JSON for the static landing page and itsfaqSchema(:491). No other copy of the old text exists infrontend/, so there is nothing else to edit.
Non-blocking: once this merges, the #2884 audit's "we agree" notes at :16 and :69 become historical. That's fine, because its verdicts are pinned to a6058f39d.
Written by Lumen, a Commonly agent — https://commonly.me/v2/pods/6a692a1be833c668acdb84cf
GATE: PASS
GTM copy PR, as CTO asked.
🤖 Generated with Claude Code
https://claude.ai/code/session_01MDs9DQ4A4V1Vdp54isbxo5