Skip to content

feat(mcp): parse validation command summaries#121

Merged
JSONbored merged 6 commits into
JSONbored:mainfrom
oktofeesh1:codex/issue-101-validation-summaries
May 30, 2026
Merged

feat(mcp): parse validation command summaries#121
JSONbored merged 6 commits into
JSONbored:mainfrom
oktofeesh1:codex/issue-101-validation-summaries

Conversation

@oktofeesh1

@oktofeesh1 oktofeesh1 commented May 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • parse structured local validation summaries for branch analysis
  • preserve status, duration, and bounded summary fields
  • treat focused/passed checks as validation evidence without sending raw logs

What changed

  • expands local validation status handling for passed, failed, skipped, focused, and unknown commands
  • sanitizes and truncates CLI validation summaries before API payloads
  • updates API/MCP schemas and PR packet rendering for duration metadata
  • keeps readiness integration fixtures current so freshness SLO assertions remain deterministic in CI

Why

  • local branch preflight should understand validation evidence even when no test files changed, while keeping logs and source content out of payloads.

Validation

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

Closes #101

Normalize local validation status, duration, and summaries before branch analysis so focused/passed checks count as evidence without sending raw logs.
@oktofeesh1
oktofeesh1 requested a review from JSONbored as a code owner May 30, 2026 05:20
@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: 4bc5c928f6

ℹ️ 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 packages/gittensory-mcp/bin/gittensory-mcp.js Outdated

@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: 3efe1fb067

ℹ️ 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 packages/gittensory-mcp/bin/gittensory-mcp.js Outdated

@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: 972dca6275

ℹ️ 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 packages/gittensory-mcp/bin/gittensory-mcp.js Outdated
Comment thread packages/gittensory-mcp/bin/gittensory-mcp.js Outdated
Comment thread packages/gittensory-mcp/bin/gittensory-mcp.js

@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: 3f36895720

ℹ️ 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 packages/gittensory-mcp/bin/gittensory-mcp.js Outdated
Comment thread packages/gittensory-mcp/bin/gittensory-mcp.js 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: validation summary parsing now covers skipped/focused/unknown cases, exit codes, durations, and path redaction.
  • Validation: API/MCP/OpenAPI surfaces and parser tests were updated together.

@dosubot dosubot Bot added the lgtm label May 30, 2026
@JSONbored
JSONbored merged commit 6e47928 into JSONbored:main May 30, 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(mcp): ingest validation command summaries v2

2 participants