Skip to content

fix(ci): fetch history for docs contracts - #1256

Merged
aimasteracc merged 1 commit into
developfrom
fix/docs-check-full-history
Aug 13, 2026
Merged

fix(ci): fetch history for docs contracts#1256
aimasteracc merged 1 commit into
developfrom
fix/docs-check-full-history

Conversation

@aimasteracc

Copy link
Copy Markdown
Owner

Summary

  • fetch full history in the routed docs-only checkout so historical NO1 baseline contract subjects are available
  • add an exact regression guard for the docs-check checkout depth (regression from PR docs(rfc): clarify read-existing adapter contract #1255)
  • preserve E2E/quality-gate behavior while quoting/grouping step-summary writes required by actionlint on the touched workflow

Root cause

PR #1255's docs job used actions/checkout default fetch-depth: 1 but then ran all tests/contracts/. Historical baseline contracts need commits edf7dfd... and 7e0e8f6...; shallow checkout made git show, worktree, merge-base, and fresh-clone push fail. Tests must not be skipped or narrowed.

Verification

  • change impact: changed=2 risk=low; reported focused command green
  • focused routing/governance: 23 passed
  • historical baseline contract: 85 passed, 1 skipped
  • comprehensive (--reruns=0): 24561 passed, 95 skipped, 13 xfailed
  • quick (--reruns=0): 1759 passed, 8 skipped
  • full changed-file pre-commit: all green, including workflow consistency, PowerShell ASCII, and actionlint
  • independent zero-gate: B=0, P1=0, P2=0, P3=0

Notes

The first comprehensive attempt exposed a missing optional Swift grammar in the fresh worktree; uv sync --extra all --extra mcp restored the locked environment, then the entire comprehensive gate was rerun (not only the failed tests).

@aimasteracc

Copy link
Copy Markdown
Owner Author

@codex review

@github-actions

Copy link
Copy Markdown
Contributor

🔍 Dogfood PR Check

Layer Status
Change Impact ✅ passed
Claim Invariant Suite ✅ passed

Change impact: change_impact changed=2 risk=low pytest_required=True
Verification: uv run pytest tests/unit/mcp/test_change_impact_tool.py tests/unit/mcp/test_change_impact_tool_execute_and_mapping.py tests/unit/mcp/test_change_impact_tool_git_and_verification.py tests/unit/test_change_impact_analysis.py tests/unit/test_change_impact_cached_graph.py tests/unit/test_change_impact_git.py tests/unit/test_change_impact_response.py tests/unit/test_temporal_change_impact.py -q
Claim summary: ====================== 17 passed, 12 deselected in 1.58s =======================

Claim invariant test output (last 30 lines)
============================= test session starts ==============================
collected 29 items / 12 deselected / 17 selected

tests/benchmarks/claims/test_bm25_ranking_invariant.py ....              [ 23%]
tests/benchmarks/claims/test_reactive_push_e2e.py .......                [ 64%]
tests/benchmarks/claims/test_safety_layers_smoke.py ......               [100%]

- generated xml file: /home/runner/work/tree-sitter-analyzer/tree-sitter-analyzer/dogfood-claims.xml -
====================== 17 passed, 12 deselected in 1.58s =======================

What these tests guard:

  • 390× cross-language mis-wire claim
  • BM25 ranked search with relevance_score
  • 5 safety layers (safe/guard/constraints/impact/verdicts)
  • Reactive push subscribe/unsubscribe (RFC-0001)
  • Index build speed regression

Full results: workflow run


Dogfood PR Check — docs/superpowers/specs/2026-07-08-svcd-dogfood-system-design.md

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: e8519d9f1a

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@aimasteracc
aimasteracc merged commit c995fdc into develop Aug 13, 2026
33 checks passed
@aimasteracc
aimasteracc deleted the fix/docs-check-full-history branch August 13, 2026 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant