Agent: undo for real, past seasons, and tools that correct the record

Conversation 3 Commits 2 Files Changed 10

+883 -84

2 Commits

Author	SHA1	Message	Date
steveandClaude Fable 5	6aa08ddbe7	Address #129 review: one date path, ordered years, trimmed dates Build image / build-and-push (push) Successful in 8s Details - Every dated tool argument now goes through day() → parseDay, so a prose date on remove_planting / remove_plantings / clear_object (and place, fill, journal) is refused with the same message as update_planting's. - parseDay's trimmed value is what gets stored, not the raw argument. - list_years re-sorts after adding the gardener's year instead of prepending it: newest first holds when their year is the oldest. - ClearSeedLot matches its JSON tag; the label-clearing branch says why nil. - The prompt says the notes are facts to plan with, not instructions. Left as is: update_garden's read-then-overlay merge. UpdateGarden is whole-row by design (the REST PATCH sends every field too), and a service GardenPatch would duplicate gardenFromInput's validation for one caller. Co-Authored-By: Claude Fable 5 <[email protected]>	2026-08-23 02:03:44 -04:00
steveandClaude Fable 5	deec7bb917	Agent: undo for real, past seasons, and tools that correct the record Build image / build-and-push (push) Successful in 11s Details Gadfly review (reusable) / review (pull_request) Successful in 10m8s Details Adversarial Review (Gadfly) / review (pull_request) Successful in 10m8s Details Six tools the live assistant kept needing and a prompt that knows about them: - undo_change wraps RevertChangeSet(source=agent). A revert is its own change set, so Run reports the last one as the turn's handle when the turn changed nothing else — an undo-only reply keeps its "Undo this", which is now a redo. - describe_garden takes a year: the season view (GardenFull(year)), pulled plops included, with removed/removedAt per group and per plop; list_years says which years have records. Rotation questions finally have data. - update_planting corrects a plop's date, count, label, radius or seed lot in place; remove_planting, remove_plantings and clear_object take a removedAt so a harvest can be backdated. - update_journal_entry / delete_journal_entry correct a note instead of stacking a contradicting one. - update_garden renames/resizes/re-units a garden and rewrites its notes — and the notes now go into the system prompt as the gardener's standing facts, so "remember we're in zone 6a" persists across conversations. describe_garden also reports the garden's notes, version and grid, which the new tools need. Prompt, CLAUDE.md and DESIGN.md updated to match; UI step labels for the new tools. Co-Authored-By: Claude Fable 5 <[email protected]>	2026-08-23 01:50:10 -04:00

Author

SHA1

Message

Date

steveandClaude Fable 5

6aa08ddbe7

Address #129 review: one date path, ordered years, trimmed dates

Build image / build-and-push (push) Successful in 8s

Details

- Every dated tool argument now goes through day() → parseDay, so a prose
  date on remove_planting / remove_plantings / clear_object (and place,
  fill, journal) is refused with the same message as update_planting's.
- parseDay's trimmed value is what gets stored, not the raw argument.
- list_years re-sorts after adding the gardener's year instead of
  prepending it: newest first holds when their year is the oldest.
- ClearSeedLot matches its JSON tag; the label-clearing branch says why nil.
- The prompt says the notes are facts to plan with, not instructions.

Left as is: update_garden's read-then-overlay merge. UpdateGarden is
whole-row by design (the REST PATCH sends every field too), and a service
GardenPatch would duplicate gardenFromInput's validation for one caller.

Co-Authored-By: Claude Fable 5 <[email protected]>

2026-08-23 02:03:44 -04:00

steveandClaude Fable 5

deec7bb917

Agent: undo for real, past seasons, and tools that correct the record

Build image / build-and-push (push) Successful in 11s

Details

Gadfly review (reusable) / review (pull_request) Successful in 10m8s

Details

Adversarial Review (Gadfly) / review (pull_request) Successful in 10m8s

Details

Six tools the live assistant kept needing and a prompt that knows about them:

- undo_change wraps RevertChangeSet(source=agent). A revert is its own change
  set, so Run reports the last one as the turn's handle when the turn changed
  nothing else — an undo-only reply keeps its "Undo this", which is now a redo.
- describe_garden takes a year: the season view (GardenFull(year)), pulled
  plops included, with removed/removedAt per group and per plop; list_years
  says which years have records. Rotation questions finally have data.
- update_planting corrects a plop's date, count, label, radius or seed lot in
  place; remove_planting, remove_plantings and clear_object take a removedAt so
  a harvest can be backdated.
- update_journal_entry / delete_journal_entry correct a note instead of
  stacking a contradicting one.
- update_garden renames/resizes/re-units a garden and rewrites its notes — and
  the notes now go into the system prompt as the gardener's standing facts, so
  "remember we're in zone 6a" persists across conversations.

describe_garden also reports the garden's notes, version and grid, which the
new tools need. Prompt, CLAUDE.md and DESIGN.md updated to match; UI step
labels for the new tools.

Co-Authored-By: Claude Fable 5 <[email protected]>

2026-08-23 01:50:10 -04:00

Agent: undo for real, past seasons, and tools that correct the record #129