Skip to content

Bump jig pin to main at PR #89 (reasoning control) - #19

Merged
cirsteve merged 1 commit into
mainfrom
chore/bump-jig-reasoning-control
Sep 10, 2026
Merged

cirsteve merged 1 commit into
mainfrom
chore/bump-jig-reasoning-control

Conversation

@cirsteve

@cirsteve cirsteve commented Sep 10, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Move the jig pin from 4fae89bb to 55081e81, the merge of Add portable reasoning switch to CompletionParams jig#89, which adds CompletionParams.reasoning (Ollama think, OpenRouter reasoning.enabled) on top of the complete-output comparison already pinned.
  • Update JIG_REVISION in scout.replay.experiments and the contract test docstring so the revision persisted on trace_comparisons matches the pin.
  • Re-render evidence/paa/reference/ and the golden fixture under tests/fixtures/paa_reference/expected/; refresh web/__tests__/fixtures/replay-worker-configuration.json. Only the embedded jig revision and the manifest hash change.
  • Fix the script path in check_reference_tree's error message (scripts/generate_paa_reference_evidence.py).

No behaviour change in scout. This makes the willie container's ephemeral jig backport unnecessary after the next deploy; wiring reasoning into sweep variants and the plan hash is a separate follow-up.

Test plan

  • uv run pytest (2435 passed, 11 skipped)
  • uv run ruff check ., uv run mypy .
  • npm test in web/ (820 passed)

🤖 Generated with Claude Code

https://claude.ai/code/session_01RzSnFgZ2rHydpunbz2b1V9

Summary by CodeRabbit

  • Maintenance

    • Updated the replay and experiment tooling to use a newer pinned Jig revision.
    • Refreshed reference evidence and validation metadata to match the latest generated artifacts.
    • Updated replay configuration and test expectations for the revised provenance information.
  • Documentation

    • Corrected guidance for regenerating missing reference evidence to point to the current script location.

Move the jig pin from 4fae89bb to 55081e81, the merge of
RankOneLabs/jig#89, which adds CompletionParams.reasoning on top of the
complete-output comparison already pinned. Re-render the PAA reference
evidence tree and the golden fixture, refresh the web replay-worker
fixture, and update JIG_REVISION and the contract test's docstring so
the persisted revision matches the pin. Also correct the script path in
check_reference_tree's error message.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RzSnFgZ2rHydpunbz2b1V9
@coderabbitai

coderabbitai Bot commented Sep 10, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Essentials

Run ID: 7d0bac1a-f180-44f8-9992-e9491692f1df

📥 Commits

Reviewing files that changed from the base of the PR and between ac58b97 and 2d0645b.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (9)
  • evidence/paa/reference/experiment-summary.json
  • evidence/paa/reference/reference-manifest.json
  • pyproject.toml
  • src/scout/paa/reference_evidence.py
  • src/scout/replay/experiments.py
  • tests/fixtures/paa_reference/expected/experiment-summary.json
  • tests/fixtures/paa_reference/expected/reference-manifest.json
  • tests/test_jig_contract.py
  • web/__tests__/fixtures/replay-worker-configuration.json

Included review availability: 2 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.


📝 Walkthrough

Walkthrough

The PR updates the pinned Jig revision across dependency, replay, test, and fixture metadata. It refreshes PAA reference hashes and updates missing-manifest guidance for the relocated evidence-generation script.

Changes

Jig and PAA synchronization

Layer / File(s) Summary
Jig revision propagation
pyproject.toml, src/scout/replay/experiments.py, tests/test_jig_contract.py, web/__tests__/fixtures/replay-worker-configuration.json
The pinned Jig revision changes to 55081e81cee5c6faf0c2d376ab4691f69e111b3d across dependency, replay, contract, and worker configuration metadata.
PAA evidence refresh and generation guidance
evidence/paa/reference/experiment-summary.json, evidence/paa/reference/reference-manifest.json, tests/fixtures/paa_reference/expected/experiment-summary.json, tests/fixtures/paa_reference/expected/reference-manifest.json, src/scout/paa/reference_evidence.py
The experiment summary and reference manifest hashes are updated in source and expected evidence. Missing-manifest guidance now references scripts/generate_paa_reference_evidence.py.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to 2d064

The pinned Jig revision and PAA reference metadata are synchronized, with the supplied test suites passing and no current correctness or deployment risk identified.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the primary change: updating the pinned Jig revision to the commit from PR #89, which adds reasoning control support.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 3 files. (6 skipped: 6 …
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/bump-jig-reasoning-control

Comment @coderabbitai help to get the list of available commands.

@cirsteve
cirsteve merged commit 58013a2 into main Sep 10, 2026
3 checks passed
@cirsteve
cirsteve deleted the chore/bump-jig-reasoning-control branch September 10, 2026 17:31
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