nsaspy/zara
1
0
Fork 0
Code Issues 95 Pull requests 18 Projects Releases Packages Wiki Activity Actions

fix: recover clean built-in todo surface toggle #264

Closed
nsaspy wants to merge 9 commits from rage/core-246-todo-toggle-refresh into master
pull from: rage/core-246-todo-toggle-refresh
merge into: nsaspy:master
Conversation 0 Commits 9 Files changed 7 +297 -127
nsaspy commented 2026-09-04 22:44:01 +00:00
Owner
Copy link

Closes #246.

Recovery of stale PR #248 onto exact current master start ed683b454bc15d14ca4fdec329640a1bf1d431d1.

Scope

  • [todo].enabled = false disables Zara's built-in todo LangChain tools, Python skills, and Prolog todo intents as one coherent surface.
  • Timer/alarm routing remains independent and available.
  • Non-boolean todo.enabled fails explicitly.
  • External providers may claim otherwise-colliding todo tool names after the built-in surface is disabled.
  • Legacy console applies the same Prolog intent toggle.

Recovery evidence

  • Source PR #248 head: 6ceb97ac7018eabc43ae1d70110d71c9be518bb3.
  • Its six implementation/test paths were untouched by all 52 commits from the stale PR base to current master, so the bounded slice was replayed without overwriting intervening Zara work.
  • Append-only recovery record: rage/246-todo-toggle-recovery.org.
  • Existing deterministic regression coverage is retained in t/test_todo_surface_toggle.py.

Fresh exact-head repository/Nix/GitHub Actions gates are required; stale #248 CI is not treated as merge evidence.

Supersedes #248.

Closes #246. Recovery of stale PR #248 onto exact current `master` start `ed683b454bc15d14ca4fdec329640a1bf1d431d1`. ## Scope - `[todo].enabled = false` disables Zara's built-in todo LangChain tools, Python skills, and Prolog todo intents as one coherent surface. - Timer/alarm routing remains independent and available. - Non-boolean `todo.enabled` fails explicitly. - External providers may claim otherwise-colliding todo tool names after the built-in surface is disabled. - Legacy console applies the same Prolog intent toggle. ## Recovery evidence - Source PR #248 head: `6ceb97ac7018eabc43ae1d70110d71c9be518bb3`. - Its six implementation/test paths were untouched by all 52 commits from the stale PR base to current master, so the bounded slice was replayed without overwriting intervening Zara work. - Append-only recovery record: `rage/246-todo-toggle-recovery.org`. - Existing deterministic regression coverage is retained in `t/test_todo_surface_toggle.py`. Fresh exact-head repository/Nix/GitHub Actions gates are required; stale #248 CI is not treated as merge evidence. Supersedes #248.
rage: record #246 exact-head RED and isolation fix
Some checks failed
CI / test (pull_request) Failing after 9s
Details
CI / android skeleton gate (pull_request) Failing after 7s
Details
CI / shared mic / Ubuntu 24.04 (pull_request) Failing after 7m25s
Details
CI / shared mic / Arch Linux (pull_request) Failing after 8m33s
Details
4769db37fb
nsaspy closed this pull request 2026-09-05 01:42:24 +00:00
Some checks failed
CI / test (pull_request) Failing after 9s
Details
CI / android skeleton gate (pull_request) Failing after 7s
Details
CI / shared mic / Ubuntu 24.04 (pull_request) Failing after 7m25s
Details
CI / shared mic / Arch Linux (pull_request) Failing after 8m33s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug
Something isn't working

  
documentation
Improvements or additions to documentation

  
duplicate
This issue or pull request already exists

  
enhancement
New feature or request

  
good first issue
Good for newcomers

  
help wanted
Extra attention is needed

  
invalid
This doesn't seem right

  
question
Further information is requested

  
wontfix
This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
nsaspy/zara!264
No description provided.