Skip to content

docs: clarify which standing approvals repeat work - #1520

Closed
santoshkumarradha wants to merge 4 commits into
devfrom
issue-1505-groom-standing-prompts
Closed

santoshkumarradha wants to merge 4 commits into
devfrom
issue-1505-groom-standing-prompts

Conversation

@santoshkumarradha

@santoshkumarradha santoshkumarradha commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

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 f855f2e6510c155b282cba388db2f92d2121dda3 on Spark: a practical shopping-note workflow, 19 generated-project tests, unchanged inventory and zero saved recurring definitions. All 30 receipts used OpenRouter deepseek/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.

Real CodeAF workflow result

Watch the workflow (MP4) · Animated GIF · Original terminal recording (.cast)

Real CodeAF workflow — speed-adjusted preview

One-off approval card

Evidence: tested revision 82e34d5dcca434536398a08c0a2849c49a4f6ae0; Spark session critical-extended-1520-v2. Execution/binary identity, model receipts, checksums and playback details. All 34 recorded calls used OpenRouter deepseek/deepseek-v4.1-flash, including auxiliary calls. Final saved standing states.

Playback and limits: Original .cast is 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-flash proposed a one-off reminder and displayed Reminds 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.

@santoshkumarradha santoshkumarradha added this to the Chat & connections milestone Sep 26, 2026
@santoshkumarradha santoshkumarradha added area:chat The v3 surface a person sits in front of (internal/tui3) sev:papercut A wording, a hint, a small wrongness that costs a moment labels Sep 26, 2026
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
santoshkumarradha force-pushed the issue-1505-groom-standing-prompts branch from 3066f9b to beabece Compare September 26, 2026 05:44
@santoshkumarradha

Copy link
Copy Markdown
Member Author

@AbirAbbas this is review ready

@santoshkumarradha
santoshkumarradha marked this pull request as draft September 27, 2026 14:42
@santoshkumarradha santoshkumarradha changed the title chat: groom the standing card ask and its yes clause docs: clarify which standing approvals repeat work Sep 27, 2026
@santoshkumarradha
santoshkumarradha marked this pull request as ready for review September 27, 2026 14:53
santoshkumarradha added a commit that referenced this pull request Sep 27, 2026
@santoshkumarradha

Copy link
Copy Markdown
Member Author

Superseded by draft #1627. Exact reviewed head 82e34d5dcca434536398a08c0a2849c49a4f6ae0 is included in published Santosh/dev history (integration b096a949c8b349a9b75d2e751783564d02890288), with remote ancestry verified and make build passing after the merge. The aggregate preserves links to this PR’s original live recording, screenshots, model receipts, checks and limitations. Closing this source PR as replaced; its branch and review/media history are retained. This does not merge it into dev, and the aggregate remains draft.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:chat The v3 surface a person sits in front of (internal/tui3) sev:papercut A wording, a hint, a small wrongness that costs a moment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants