Skip to content

feat(preflight): generate public-safe pr packets#53

Merged
JSONbored merged 1 commit into
JSONbored:mainfrom
oktofeesh1:codex/public-safe-pr-packets
May 29, 2026
Merged

feat(preflight): generate public-safe pr packets#53
JSONbored merged 1 commit into
JSONbored:mainfrom
oktofeesh1:codex/public-safe-pr-packets

Conversation

@oktofeesh1

@oktofeesh1 oktofeesh1 commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Generate a public-safe PR packet from local branch metadata only.
  • Expose packet markdown in the local branch analysis response and MCP CLI packet output.

What changed

  • Added prPacket.markdown with sections for summary, linked context, branch freshness, overlap/WIP checks, changed paths, validation, and next steps.
  • Redacted local absolute paths from changed-file summaries and filtered sensitive contributor/review language from packet lines.
  • Added a final local CLI sanitizer before non-JSON packet output prints server-provided markdown.
  • Added regression coverage that rejects score, reward, wallet, hotkey, raw-trust, and private-reviewability language from non-JSON packet output.
  • Updated the MCP package changelog under Unreleased.

Why

Contributors need a copy/paste PR packet that is safe to publish without exposing local machine paths, sensitive review context, or source contents. This keeps the workflow metadata-only and avoids accidental source upload while still giving maintainers the freshness, overlap, and validation context they need.

Closes #39.

Validation

  • npm run test:ci
  • npm run changelog:check:mcp
  • GitHub validate check triggered on the latest pushed commit.

Notes

  • Source upload remains disabled for this workflow.
  • The packet is generated from local git and preflight metadata, not raw file contents.
  • Package version remains 0.2.0; MCP package version bumps are release-tag driven through the mcp-v* publish workflow, while this PR records the CLI surface change in the package changelog.

@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.

PR #53 — request changes
feat(preflight): generate public-safe pr packets

Required changes:

  • Wait for/trigger validate; GraphQL shows no validate check on the latest commit.
  • Add a final local CLI sanitizer before printing server-provided packet markdown in non-JSON mode.
  • Test that non-JSON packet output rejects score/reward/wallet/hotkey/raw-trust/private-reviewability language even if a fixture/server payload regresses.
  • Confirm package changelog/version policy because this changes the npm CLI surface.

Add metadata-only PR packet markdown for local branch analysis and expose it through the MCP CLI packet flow. Include branch freshness, overlap caution, changed-path summaries, validation, and tests for public-safe redaction.
@oktofeesh1
oktofeesh1 force-pushed the codex/public-safe-pr-packets branch from dba8eac to 8167b95 Compare May 29, 2026 07:35
@JSONbored
JSONbored dismissed their stale review May 29, 2026 07:37

issues resolved

@JSONbored
JSONbored merged commit 3ace069 into JSONbored:main May 29, 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(preflight): generate public-safe PR packets from metadata only

2 participants