docs: pre-1.0 patch judgement note + Pi model footer matrix fix#469
Open
Nathan Schram (nathanschram) wants to merge 3 commits intodevfrom
Open
docs: pre-1.0 patch judgement note + Pi model footer matrix fix#469Nathan Schram (nathanschram) wants to merge 3 commits intodevfrom
Nathan Schram (nathanschram) wants to merge 3 commits intodevfrom
Conversation
All 9 v0.35.3 Group 2 issues now landed on dev: - #404 — secret-scanning alert (PR #439) - #297 — /trigger → /listen rename + alias (PR #440) - #294 — master trigger pause/resume toggle (PR #441) - #380 — auto-approve scope review (PR #442) - #438 — claude_stream_idle_timeout_ms + Type-A/B classification (PR #443) - #410 — subscription usage observability + /usage debug (PR #444) - #271 — trigger visibility Tier 2 + Tier 3 (PR #445) - #333 — Claude post-result idle timeout + ✓ turn complete UX hint (PR #446) - #269 — hot-reload [progress] settings (PR #447) Bumps to TestPyPI for staging via @hetz_lba1_bot once integration tests U1-U7 pass against @untether_dev_bot. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Two small docs-only updates: - .claude/rules/release-discipline.md — add "Pre-1.0 minor judgement" paragraph clarifying that opt-in additive features behind config flags may ship in patches if they preserve all current defaults. Codifies precedent set by [security] env_extra_allow (#409 in v0.35.3). Removes ambiguity for upcoming v0.35.8 (Pi RPC, #170) decision. - README.md — fix engine compatibility matrix: Pi "Model in footer" cell changes from "—" to "✅⁷" with footnote explaining Pi populates the footer model from a supplementary StartedEvent carrying the model name extracted from message_end (#225). Code at src/untether/runners/pi.py:290-307 has implemented this since #225 landed; matrix was stale. Related: docs/plans/2026-05-02-pi-engine-enhancements.md (local plan, gitignored) — covers the broader Pi enhancement roadmap that motivated both fixes. Issues created: #460-#468 + promotions of #170 → v0.35.8 and #180 → v0.35.9. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
- docs/reference/dev-instance.md: add Engine auth & defaults section (matrix of CLI ver, auth method, default provider/model, cred location for Claude/Codex/Gemini/OpenCode/Pi/AMP), smoke verification commands, per-engine notes (OpenCode google-whitelist=[], Pi Kimi K2.6 zero-cost, Gemini trusted folders + #471, AMP mode-not-model routing). Also document the three special-purpose instances (demo, dev-hf, dev-ws) and the lockfile PID-reuse race - docs/reference/integration-testing.md: cross-reference the new auth section so test runs check current state before tier 1 - CLAUDE.md, .claude/rules/dev-workflow.md: extend 2-instance section to the full 5-instance topology, clarifying that demo/dev-hf/dev-ws share the editable .venv but are not on the release path - .claude/hooks.json: dev-workflow-guard now recognises all 4 editable-source services (untether-dev, untether-dev-hf, untether-dev-ws, untether-demo) and only blocks restarts of staging Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.claude/rules/release-discipline.mdclarifying when opt-in additive features can ship in patches vs minors. Codifies precedent already in use (#409 shipped a config addition in a patch).README.md: Pi Model in footer cell changes from—to✅⁷with footnote 7 explaining the supplementaryStartedEventmechanism. Code atsrc/untether/runners/pi.py:290-307already does this (since #225); the matrix was stale.Why
Both fixes surfaced during the Pi engine audit that produced the upcoming roadmap:
--mode rpc) #170 → v0.35.8, feat: Pi extension bridge — lifecycle hooks via extension system #180 → v0.35.9The judgement-note edit removes ambiguity for the v0.35.8 RPC runner decision (#170). The matrix fix is a pure code-doc reconciliation — no functional change.
Test plan
git diff --stat→ 2 files, 4 insertions, 1 deletionNotes
docs/plans/2026-05-02-pi-engine-enhancements.md(local —docs/plans/is gitignored).untether-outbox/pi-issue-drafts.mdfor the full issue body drafts that were created as bug(pi): wire AutoRetry events (schema-defined but never handled) #460-feat(pi): /providers Telegram command — list providers + auth status #468🤖 Generated with Claude Code