From 6328ed884c1b13ad15a83e074a7314495612064d Mon Sep 17 00:00:00 2001 From: Ben Sima Date: Mon, 1 Dec 2025 07:54:07 -0500 Subject: Improve Worker.hs prompt to avoid redundant test/lint runs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Perfect! The changes are exactly what we need. The diff shows that I've 1. ✅ `bild --test` on a namespace tests all its dependencies - no need t 2. ✅ Don't re-run tests after they already passed 3. ✅ Lint runs automatically via git hooks, so manual runs are optional I have successfully completed task t-202. The changes made to `Omni/Agen 1. **Explaining dependency handling**: Added explicit note that `bild -- 2. **Preventing redundant testing**: Added instruction to not re-run tes 3. **Clarifying lint automation**: Explained that lint runs automaticall All tests pass (`bild --test Omni/Agent/Worker.hs`) and all lint checks Task-Id: t-202 --- Omni/Agent/Worker.hs | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'Omni/Agent/Worker.hs') diff --git a/Omni/Agent/Worker.hs b/Omni/Agent/Worker.hs index a34270c..cc5f730 100644 --- a/Omni/Agent/Worker.hs +++ b/Omni/Agent/Worker.hs @@ -325,6 +325,14 @@ buildBasePrompt task ns repo = <> "7. Only exit after bild --test passes.\n\n" <> "IMPORTANT: The git commit will fail if lint finds issues.\n" <> "You must fix all lint suggestions.\n\n" + <> "BUILD SYSTEM NOTES:\n" + <> "- Running 'bild --test " + <> ns + <> "' automatically tests ALL dependencies of that namespace\n" + <> "- You do NOT need to run bild --test on individual files - just the main namespace\n" + <> "- Once tests pass, do NOT re-run them unless you make more changes\n" + <> "- The 'lint' command will be run automatically during git commit via hooks\n" + <> "- You can run 'lint --fix' on changed files if needed, but it's optional\n\n" <> "Context:\n" <> "- Working directory: " <> Text.pack repo -- cgit v1.2.3