Skip to content

feat(github-agent): add public-safe answer cards - #192

Merged
JSONbored merged 2 commits into
JSONbored:mainfrom
oktofeesh1:codex/github-agent-answer-cards
Jun 1, 2026
Merged

feat(github-agent): add public-safe answer cards#192
JSONbored merged 2 commits into
JSONbored:mainfrom
oktofeesh1:codex/github-agent-answer-cards

Conversation

@oktofeesh1

Copy link
Copy Markdown
Contributor

Summary

  • add a reusable public answer-card contract for @gittensory GitHub replies
  • add source/freshness notes and collapsible safe-details sections
  • extend public sanitizer coverage for scoreability, public score estimates, and private reviewability language

What changed

  • Wrapped command-specific reply content in a shared card shape with summary, findings, evidence, next actions, and source/freshness notes.
  • Keeps long safe details in collapsible sections so public comments stay concise.
  • Runs card fields through the public sanitizer before posting and again at final comment assembly.
  • Adds regression coverage for card sections, citation/source notes, refresh and empty fallbacks, safe details, duplicate-line dedupe, and forbidden public-output terms.

Why

Closes #141. Public GitHub replies now have one consistent contract instead of command-specific formatting, while private scorer and planning context remains out of public comments.

Validation

  • npm run typecheck
  • npm run test:unit -- test/unit/github-commands.test.ts
  • npm run test:coverage
  • npm run test:ci
  • Codex Security diff scan: 2/2 worklist rows reviewed, no reportable findings.

Notes

  • This PR keeps the existing command set and authorization model unchanged.
  • It is intentionally separate from the command-router v2 PR so the answer-card contract can be reviewed on its own.

@oktofeesh1
oktofeesh1 force-pushed the codex/github-agent-answer-cards branch from 1f41184 to 40e8e8b Compare June 1, 2026 18:49

@JSONbored JSONbored left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

@oktofeesh1 This is ready from my side. The public-safe answer-card work is green and merge-clean.

A few notes:

  • The GitHub command output shape is covered by tests.
  • The formatting keeps public GitHub responses structured without exposing private maintainer context.
  • No requested changes from me.

@dosubot dosubot Bot added the lgtm label Jun 1, 2026
@JSONbored
JSONbored merged commit 827c7bb into JSONbored:main Jun 1, 2026
3 checks passed
@JSONbored JSONbored added gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier. and removed feature labels Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(github-agent): add public-safe answer cards with citations

2 participants