Skip to content

Keep approved fix-later items in every status update - #739

Merged
mergify[bot] merged 1 commit into
mainfrom
stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4
Sep 17, 2026
Merged

mergify[bot] merged 1 commit into
mainfrom
stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4

Conversation

@EdbertChan

@EdbertChan EdbertChan commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Summary

The agent follows a set of house rules for how it works with you. One rule now covers work you agreed to put off.

Say you tell the agent to fix a thing later. Its next update lists only what got done, so the put-off thing drops out of sight.

Now the agent keeps it on the list. Every update and wrap-up shows it, marked still open. It stays until the work is done or you drop it.

Review Claim

The learned working-style rules keep approved deferred items visible until closed.

Review Lane

behavior

Review Unit

corpus-lesson

Safety Invariant

Adds one rule; nothing else changes.

Slice Rationale

One rule and its test. No known prior art is cited.

Non-goals

  • The test pins the rule text only.

Test Plan

Test Plan
  • python3 -m unittest tests.test_deferred_items_rule: fails before, passes after
  • Every step in .github/workflows/ci.yml ran locally on the full stack with rc=0: run_all_tests.sh, check_no_tracked_local_artifacts, check_hook_test_coverage, check_no_silent_hook_except, check_rules_are_wired, check_skills_three_harnesses, check_ecosystem_boundaries, check_skill_file_refs, check_skill_test_coverage, check_skill_test_debt_no_growth, check_skill_trigger_mechanism, check_dora_baseline, check_no_dated_provenance, check_no_new_comments, ruff check . --select E9,F, shellcheck install.sh

Revert Plan

Revert Plan
  • Safe to revert? Yes
  • Revert command: git revert <sha>
  • Post-revert steps: rerun ./install.sh so installed copies match
  • Data migration? No

🤖 Generated with Claude Code

@EdbertChan

EdbertChan commented Sep 17, 2026

Copy link
Copy Markdown
Owner Author

This pull request is part of a Mergify stack:

# Pull Request Link
1 Keep approved fix-later items in every status update #739 👈
2 Search saved memory both ways before a deep dive #740

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from 6616ba2 to 6de7950 Compare September 17, 2026 00:49
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/never-resolve-conflict-taking-one-side-s-whole--6c4fab6a branch from 047448a to 5b27b14 Compare September 17, 2026 00:49
@EdbertChan

EdbertChan commented Sep 17, 2026

Copy link
Copy Markdown
Owner Author

Revision history

# Type Changes Reason Date
1 initial 6616ba2 2026-09-17 00:49 UTC
2 content 6616ba2 → 6de7950 2026-09-17 00:49 UTC
3 rebase 6de7950 → cf6cf76 (rebase only) 2026-09-17 04:23 UTC
4 rebase cf6cf76 → edebaff (rebase only) 2026-09-17 04:37 UTC
5 rebase edebaff → 07a35ef (rebase only) 2026-09-17 04:55 UTC
6 rebase 07a35ef → a956a8a (rebase only) 2026-09-17 05:11 UTC
7 rebase a956a8a → d4318fe (rebase only) 2026-09-17 05:26 UTC
8 rebase d4318fe → 627512e (rebase only) 2026-09-17 05:40 UTC

@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from 6de7950 to cf6cf76 Compare September 17, 2026 04:23
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/never-resolve-conflict-taking-one-side-s-whole--6c4fab6a branch from 5642a01 to 462018d Compare September 17, 2026 04:37
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from cf6cf76 to edebaff Compare September 17, 2026 04:37
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/never-resolve-conflict-taking-one-side-s-whole--6c4fab6a branch from 462018d to ef089ee Compare September 17, 2026 04:55
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from edebaff to 07a35ef Compare September 17, 2026 04:55
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/never-resolve-conflict-taking-one-side-s-whole--6c4fab6a branch from ef089ee to bf3fe1e Compare September 17, 2026 05:11
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from 07a35ef to a956a8a Compare September 17, 2026 05:11
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/never-resolve-conflict-taking-one-side-s-whole--6c4fab6a branch from bf3fe1e to 937de7d Compare September 17, 2026 05:25
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from a956a8a to d4318fe Compare September 17, 2026 05:25
A recap that lists only finished work makes the deferred half disappear. Add
the learned rule and a test that pins it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Change-Id: Ifec2cad4fd573aee970dd3c6b7839cd19192302a
@EdbertChan
EdbertChan changed the base branch from stack/EdbertChan/feat/memory-rules-into-catstack/never-resolve-conflict-taking-one-side-s-whole--6c4fab6a to main September 17, 2026 05:40
@EdbertChan
EdbertChan force-pushed the stack/EdbertChan/feat/memory-rules-into-catstack/keep-approved-fix-later-items-status-update--fec2cad4 branch from d4318fe to 627512e Compare September 17, 2026 05:40
@mergify

mergify Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Queued — the merge queue status continues in this comment ↓.

@EdbertChan

Copy link
Copy Markdown
Owner Author

@Mergifyio queue

@mergify

mergify Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 7 minutes 9 seconds in the queue, including 6 minutes 44 seconds running CI.

Required conditions to merge
  • check-success = lint
  • check-success = test
  • check-success = validate

@mergify mergify Bot added the queued label Sep 17, 2026
@mergify
mergify Bot merged commit d4e6307 into main Sep 17, 2026
7 of 12 checks passed
@mergify mergify Bot removed the queued label Sep 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant