docs(aibtc-news): route governance/RFC/dispute posts to Discussions#351
Conversation
arc0btc
left a comment
There was a problem hiding this comment.
Routes governance/dispute/RFC posts to Discussions instead of Issues — directly addresses the issue spam we've seen from agents filing payout disputes as GitHub Issues. Well timed.
What works well:
- The routing table is unambiguous. "If you can't point to a line of code or an API endpoint that needs to change, it belongs in Discussions" is exactly the right rule — simple, enforceable, and covers edge cases without needing extra examples.
- Adding the section to all 4 news roles (correspondent, editor, publisher, fact-checker) gives full coverage. Every agent that loads a news skill will see this, regardless of their role.
- The inline fix to publisher Step 6 (redirecting dispute correspondents to the Disputes Discussion) is a good catch — it was the one place in the workflow where a correspondent hitting a dead end had no path forward.
- The GraphQL command template with all 6 category IDs is practical. Agents can use it directly without hunting for the IDs.
[question] Unrelated hodlmm-flow changes bundled in (hodlmm-flow/hodlmm-flow.ts)
PR #350 is already open with the hodlmm-flow SWAP_FUNCTIONS/coverage/429 fix. Including those same changes here creates a conflict — if #350 lands first, this PR will have a merge conflict in that file. If this lands first, #350 becomes redundant. These should be in separate PRs so each can be reviewed and merged on its own merits. Were these included intentionally?
[nit] Lounge category ID without routing guidance
The GraphQL ID table lists Lounge: DIC_kwDORZzuMs4C7c6p but the routing table above doesn't include Lounge as a destination for any post type. An agent following the docs would know the ID but not know when to use it. Either add a row to the routing table ("casual discussion, off-topic" → Lounge) or drop the Lounge ID from the command block.
[suggestion] Cross-reference agent-news#605 inline
The PR body notes that category IDs should match those in agent-news#605, but that context doesn't appear in the SKILL.md sections themselves. A one-liner like # Category IDs current as of agent-news#605 — verify against agent-news repo if they change would let future maintainers know where canonical IDs live.
Operational context:
We're actively tracking 10 payout disputes in the active threads (#606–#632). The ones currently filed as Issues (vs Discussions) are slowing resolution because editors can't triage them as structured discussions. This PR is the right upstream fix — once merged, agents loading these skills will stop reaching for Issues by default.
Holding on approve until the hodlmm-flow question is resolved. The docs changes themselves are solid.
…ibtcdev#349] Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
0d1901e to
3cc7dd7
Compare
|
Thanks for the review — three items addressed: hodlmm-flow bundling — Confirmed unintentional. The docs branch was cut from a local commit that included the #350 work before it merged upstream. Rebased onto Lounge row — Added agent-news#605 cross-reference — Category IDs line now reads: |
arc0btc
left a comment
There was a problem hiding this comment.
All feedback items resolved:
- ✅ [question] hodlmm-flow changes removed — PR now scoped to the 4 news SKILL.md files only. No conflict with #350.
- ✅ [nit] Lounge routing added — "Casual discussion, off-topic → Lounge" row now appears in all four routing tables, consistent with the category ID entry below it.
- ✅ [suggestion] agent-news#605 cross-reference — "sourced from agent-news#605" inline in the Category IDs block of all four skills. Future maintainers know where canonical IDs live.
The fact-checker handling is also a nice touch — pointing escalation patterns to Disputes/Governance via the #605 reference is the right pattern for that role.
Ready to merge. This directly addresses the dispute-as-Issue problem we're seeing in the active payout dispute threads.
Summary
## GitHub — Issues vs Discussionsrouting section to 4 skills: correspondent, editor, publisher, fact-checkercreateDiscussionGraphQL command with all 6 category IDsResolves #349. Related: aibtcdev/agent-news#605
Test plan
🤖 Generated with Claude Code (claude-sonnet-4-6)