Agent: catalog and garden tools, and a ready date on every describe group

Conversation 3 Commits 2 Files Changed 7

+313 -19

2 Commits

Author	SHA1	Message	Date
steveandClaude Fable 5	c9076e84c4	Address #130 review: one toolCaller helper for the tool tests Build image / build-and-push (push) Successful in 7s Details The call/mustCall closures were copied between TestRecordKeepingTools and TestCatalogAndGardenTools; both now use a file-level toolCaller. The other notes are left as they are: the 'nothing to change' guard enumerates the args on purpose (it is the tool's own contract, next to the struct it checks), and wrapping a sentinel with %w is how every readable refusal in this package is built. Co-Authored-By: Claude Fable 5 <[email protected]>	2026-08-23 02:11:27 -04:00
steveandClaude Fable 5	b4c8007977	Agent: catalog and garden tools, and a ready date on every describe group Build image / build-and-push (push) Successful in 11s Details Gadfly review (reusable) / review (pull_request) Successful in 4m42s Details Adversarial Review (Gadfly) / review (pull_request) Successful in 4m42s Details - update_seed_lot / delete_seed_lot: correct or drop a recorded purchase ("it was three packets, not two"); the plant a lot is for stays fixed. - delete_plant: remove a duplicate from the user's catalog. The service already refuses while plantings (past seasons included) or a lot reference it; the tool turns that sentinel into words the model can pass on, and tells it not to clear those references to get its way. - create_garden: a new place, with the service's defaults; the prompt says a plan is still a copy_garden. - describe_garden groups carry readyAround — planting date plus days to maturity for the plops still in the ground — so "what can I pick this week?" is a lookup rather than arithmetic the model got wrong live. Co-Authored-By: Claude Fable 5 <[email protected]>	2026-08-23 02:05:19 -04:00

Author

SHA1

Message

Date

steveandClaude Fable 5

c9076e84c4

Address #130 review: one toolCaller helper for the tool tests

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

Details

The call/mustCall closures were copied between TestRecordKeepingTools and
TestCatalogAndGardenTools; both now use a file-level toolCaller. The other
notes are left as they are: the 'nothing to change' guard enumerates the
args on purpose (it is the tool's own contract, next to the struct it
checks), and wrapping a sentinel with %w is how every readable refusal in
this package is built.

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

2026-08-23 02:11:27 -04:00

steveandClaude Fable 5

b4c8007977

Agent: catalog and garden tools, and a ready date on every describe group

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

Details

Gadfly review (reusable) / review (pull_request) Successful in 4m42s

Details

Adversarial Review (Gadfly) / review (pull_request) Successful in 4m42s

Details

- update_seed_lot / delete_seed_lot: correct or drop a recorded purchase
  ("it was three packets, not two"); the plant a lot is for stays fixed.
- delete_plant: remove a duplicate from the user's catalog. The service
  already refuses while plantings (past seasons included) or a lot reference
  it; the tool turns that sentinel into words the model can pass on, and
  tells it not to clear those references to get its way.
- create_garden: a new place, with the service's defaults; the prompt says a
  plan is still a copy_garden.
- describe_garden groups carry readyAround — planting date plus days to
  maturity for the plops still in the ground — so "what can I pick this
  week?" is a lookup rather than arithmetic the model got wrong live.

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

2026-08-23 02:05:19 -04:00

Agent: catalog and garden tools, and a ready date on every describe group #130