Skip to content

feat(mcp): add current-branch GitHub status hints - #122

Merged
JSONbored merged 5 commits into
JSONbored:mainfrom
oktofeesh1:codex/issue-102-ci-hints
May 30, 2026
Merged

feat(mcp): add current-branch GitHub status hints#122
JSONbored merged 5 commits into
JSONbored:mainfrom
oktofeesh1:codex/issue-102-ci-hints

Conversation

@oktofeesh1

@oktofeesh1 oktofeesh1 commented May 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • bind cached GitHub PR status to current local branch analysis
  • surface approved, pending, failing, and needs-author branch states
  • include public-safe GitHub status context in generated PR packets

What changed

  • matches current branch metadata by head SHA or branch ref
  • adds branch findings for failing checks, requested changes, and incomplete status data
  • tests PR-found, no-PR, failing status, approved review, and unavailable-status fallback cases
  • keeps readiness integration fixtures current so freshness SLO assertions remain deterministic in CI

Why

  • branch preflight should explain already-known GitHub review/check state before contributors ask maintainers for review.

Validation

  • git diff --check
  • npx tsc --noEmit
  • npx vitest run test/unit/local-branch.test.ts
  • npm run test:coverage
  • GitHub validate, Contributor trust, and Superagent Security Scan pass on the latest head

Closes #102

Bind cached GitHub PR review and check state to local branch analysis so blockers and public-safe packets include current PR status context.
@oktofeesh1
oktofeesh1 requested a review from JSONbored as a code owner May 30, 2026 05:21
@dosubot dosubot Bot added the size:M label May 30, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c5311f5f5c

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/signals/local-branch.ts Outdated
Comment thread src/signals/local-branch.ts
Comment thread src/signals/local-branch.ts
Comment thread src/signals/local-branch.ts Outdated
@dosubot dosubot Bot added size:L and removed size:M labels May 30, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4b2b4bb4a7

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/signals/local-branch.ts
Comment thread src/signals/local-branch.ts Outdated
Comment thread src/signals/local-branch.ts Outdated
Comment thread src/signals/local-branch.ts Outdated

@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 merge-ready from my pass.

  • Deterministic: check rollup is green and merge-tree against current origin/main is clean.
  • Maintainer judgment: current-branch GitHub status is now wired into local branch analysis, MCP/API paths, PR packet output, and schema coverage.
  • Validation: tests cover branch matching, base-ref matching, failing/pending checks, drafts, requested changes, and public-safe packet output.

@dosubot dosubot Bot added the lgtm label May 30, 2026
@JSONbored
JSONbored merged commit b640cb9 into JSONbored:main May 30, 2026
3 checks passed
@JSONbored JSONbored added gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. gittensor:feature Gittensor-scored feature linked to a feature issue — scores a 0.25x multiplier. and removed feature gittensor:bug Gittensor-scored bug fix — scores a 0.05x multiplier. 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(mcp): ingest GitHub CI status hints for local branches

2 participants