Skip to content

feat(scoring): derive observed pending PR scenarios - #57

Merged
JSONbored merged 2 commits into
JSONbored:mainfrom
oktofeesh1:codex/pending-pr-scenarios
May 29, 2026
Merged

feat(scoring): derive observed pending PR scenarios#57
JSONbored merged 2 commits into
JSONbored:mainfrom
oktofeesh1:codex/pending-pr-scenarios

Conversation

@oktofeesh1

@oktofeesh1 oktofeesh1 commented May 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Derive observed pending-PR scenarios from cached GitHub pull request state.
  • Keep user-supplied assumptions separate from GitHub-observed state in private branch analysis.
  • Preserve the public packet boundary while adding regression coverage for the new metadata path.

What changed

  • Persist draft, mergeability, and review-decision fields from the open-PR GraphQL supplement.
  • Add private scenarios for approved or mergeable PRs landing and stale or closed PRs clearing.
  • Wire contributor-wide cached PR context into local branch API, MCP, and agent analysis.
  • Extend OpenAPI schemas and focused unit coverage for persistence, scenario classification, and public packet sanitization.

Why

Closes #17. Local branch analysis can now model pending pull request state from cached GitHub data instead of relying only on manually supplied assumptions.

Validation

  • npx vitest run test/unit/backfill.test.ts test/unit/db-parsers.test.ts test/unit/local-branch.test.ts test/unit/scoring.test.ts
  • npm run test:ci
  • git diff --check
  • Codex Security diff scan completed locally with no reportable security findings.

Notes

  • Scenario output remains private API/MCP context.
  • Public PR packet sanitization continues to reject private account-sensitive language.

@oktofeesh1
oktofeesh1 requested a review from JSONbored as a code owner May 29, 2026 08:46
@dosubot dosubot Bot added the size:L label May 29, 2026
@JSONbored JSONbored self-assigned this May 29, 2026
@JSONbored JSONbored assigned oktofeesh1 and unassigned JSONbored May 29, 2026

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

#57 feat(scoring): derive observed pending PR scenarios
action: merge
required:

  • None obvious. Good scoreability projection work with validate passing.

@dosubot dosubot Bot added the lgtm label May 29, 2026
@JSONbored
JSONbored merged commit 8e714a7 into JSONbored:main May 29, 2026
3 checks passed
jony376 pushed a commit to jony376/gittensory that referenced this pull request May 31, 2026
Co-authored-by: ghost <49853598+JSONbored@users.noreply.github.com>
@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(scoring): detect approved and mergeable pending PR scenarios

2 participants