docs: clarify which standing approvals repeat work - #1520
Closed
santoshkumarradha wants to merge 4 commits into
Closed
santoshkumarradha wants to merge 4 commits into
santoshkumarradha wants to merge 4 commits into
Conversation
The lead said "wants to keep an eye on" on every standing card, which was wrong about most of them: a one-off reminder watches nothing, and a rule that never wakes watches nothing either. The lead now says what a yes binds — "wants to set this up" — for every kind. The yes clause beside the chip promised "it keeps happening until you stop it" on cards whose item runs once at a moment and then retires. The clause now reads the item: at-things get "it happens at the time, and then it retires", and every other kind keeps the old clause. The manual pages that quote the card move with it, and the tests that assert the lead's words say the new words.
santoshkumarradha
force-pushed
the
issue-1505-groom-standing-prompts
branch
from
September 26, 2026 05:44
3066f9b to
beabece
Compare
Member
Author
|
@AbirAbbas this is review ready |
santoshkumarradha
marked this pull request as draft
September 27, 2026 14:42
santoshkumarradha
marked this pull request as ready for review
September 27, 2026 14:53
This was referenced Sep 27, 2026
santoshkumarradha
added a commit
that referenced
this pull request
Sep 27, 2026
Member
Author
|
Superseded by draft #1627. Exact reviewed head |
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
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.
Supplemental correction verified in separate PR #1612
Separate PR #1612 adds task-aware one-off wording: Run it then / Runs the work then, while retaining reminder-specific text. The live recording shows the corrected work card and its decline.
Tested revision
f855f2e6510c155b282cba388db2f92d2121dda3on Spark: a practical shopping-note workflow, 19 generated-project tests, unchanged inventory and zero saved recurring definitions. All 30 receipts used OpenRouterdeepseek/deepseek-v4.1-flash; Model Pool was disabled as disclosed in coverage/limitations.Fix PR #1612 · Video · GIF · Original terminal recording · Actual corrected card · Post-approval execution evidence
This evidence tests the separate corrective revision; it does not claim this PR’s original head already contains that fix. Earlier recordings and their limitations remain below. Tomorrow’s task was declined, not executed; ordinary model-action and permission limits still apply. Integration boundary: this #1612 base does not contain #1560 does.isolate, so ordinary Once tools do not prove isolated execution. Consolidation must suppress Once for isolated tasks or use a shared isolated executor. Isolated Once and watch/probe Once were not live-verified.
Real product workflow — recorded on Spark
Goal: Prepare a practical pantry shopping plan and choose the right ongoing commitments: a one-off reminder, a weekly reminder and a standing permission rule.
Observed outcome: The app wrote shopping-plan.md. The one-off reminder fired and retired; the weekly reminder was approved then stopped through the product; the permission rule remained active without a wake. The cards and stored states distinguish one-off, repeating and always-on commitments.
Watch the workflow (MP4) · Animated GIF · Original terminal recording (.cast)
Evidence: tested revision
82e34d5dcca434536398a08c0a2849c49a4f6ae0; Spark sessioncritical-extended-1520-v2. Execution/binary identity, model receipts, checksums and playback details. All 34 recorded calls used OpenRouterdeepseek/deepseek-v4.1-flash, including auxiliary calls. Final saved standing states.Playback and limits: Original
.castis preserved; GIF/MP4 play at 3× speed with idle intervals capped at 3 seconds. Run used --one-model and explicitly disabled Model Pool for issue #1608; Model Pool behavior is verified separately in #1610. Earlier failed-policy recordings remain preserved locally and are not counted as successful evidence. The model briefly described an already-approved weekly reminder as pending; a user correction and listing resolved it. The recording keeps that rough edge. The PR is documentation-only; this demonstrates the documented current product behavior, not a new implementation of reminder labels.Explain the different engine-owned consequences of accepting a reminder, rule or repeating check. The manual quotes the labels and consequences the current cards actually display.
Related to #1505. This PR changes documentation only; the current engine already gives one-off reminders a non-repeating consequence.
Validation: live tmux with OpenRouter
deepseek/deepseek-v4.1-flashproposed a one-off reminder and displayedReminds you then. Nothing repeats.The proposal was declined after inspection. All auxiliary-call receipts named the same model. GitHub light gate, touched packages, aggregate check and CLA all passed.