summaryrefslogtreecommitdiff
path: root/Omni
AgeCommit message (Collapse)Author
2025-11-24deprecate(agent): move functionality to jrBen Sima
Moves the 'agent start' and 'agent merge-driver' commands to 'jr work' and 'jr merge-driver' respectively. Deletes Omni/Agent.hs as it is no longer the entry point. Amp-Thread-ID: https://ampcode.com/threads/T-ac41b9b6-d117-46de-9e4f-842887a22f1d Co-authored-by: Amp <amp@ampcode.com>
2025-11-24fix(agent): round credits to 2 decimal places and use totalCreditsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ac41b9b6-d117-46de-9e4f-842887a22f1d Co-authored-by: Amp <amp@ampcode.com>
2025-11-24agent: add note about legacy merge driver logicBen Sima
Adds a comment to the merge driver logic in Omni/Agent.hs acknowledging that it may be obsolete given the removal of git-tracked tasks.jsonl, but keeping it for now to avoid breaking potential legacy rebase scenarios. Amp-Thread-ID: https://ampcode.com/threads/T-ac41b9b6-d117-46de-9e4f-842887a22f1d Co-authored-by: Amp <amp@ampcode.com>
2025-11-24feat: implement t-1o2egbj8o0n.3Ben Sima
I have implemented the `jr work` command in `Omni/Jr.hs`. **Changes:** 1. **Modified `Omni/Jr.hs`**: * Added necessary imports (`Omni.Agent.Core`, `Omni.Agent.Worker`, `System.Directory`, `System.FilePath`, `Data.Text`). * Updated the Docopt usage to include `jr work [<task-id>]`. * Implemented the `work` command handler in the `move` function to initialize a `Worker` and call `Worker.start`, mirroring the logic in `Omni/Agent.hs`. * Added unit tests to verify parsing of the `work` command with and without a task ID. **Verification:** * Ran `bild --test Omni/Jr.hs` which passed, confirming the code compiles and tests pass. * Ran `lint Omni/Jr.hs` which passed. * Verified `_/bin/jr --help` shows the new command. I am now ready to exit.
2025-11-24Remove harvest command and documentationBen Sima
The 'harvest' functionality was tied to git-synced JSONL task files, which have been replaced by a local SQLite database. This commit removes the command from the CLI and updates documentation to reflect the new workflow. Amp-Thread-ID: https://ampcode.com/threads/T-ac41b9b6-d117-46de-9e4f-842887a22f1d Co-authored-by: Amp <amp@ampcode.com>
2025-11-24feat: implement t-1o2egbj8o0n.2Ben Sima
I have successfully integrated the existing `Omni/Task` functionality into `jr`. The `Omni/Jr.hs` file already had the logic to forward `task` commands to `Omni/Task.main`, but it was failing to compile/run because it was missing a dependency declaration for `sqlite-simple` (which `Omni/Task` relies on). I added the missing dependency to `Omni/Jr.hs`: ```haskell -- : dep sqlite-simple ``` I verified the fix by running: 1. `Omni/Ide/run.sh Omni/Jr.hs task list` - Verified it lists tasks correctly. 2. `Omni/Ide/run.sh Omni/Jr.hs task create "Test Task via Jr"` - Verified it creates tasks. 3. `Omni/Ide/run.sh Omni/Jr.hs task show <id>` - Verified it shows task details. 4. `Omni/Ide/run.sh Omni/Jr.hs task -h` - Verified it shows `task` specific help. 5. `Omni/Ide/run.sh Omni/Jr.hs task test` - Verified it runs the `Omni/Task` test suite. 6. `bild --test Omni/Jr.hs` - Verified `Omni/Jr.hs` itself builds and passes its own tests. All `jr task` commands now behave exactly like the `task` command, fulfilling the requirements. The existing `Omni/Task` logic is fully reused.
2025-11-24agent: restore git commit with amp outputBen Sima
Re-enables git commits in the worker, using the captured output from 'amp' as the commit message. Also updates 'Omni/Agent.hs' to handle the API change in TaskCore.exportTasks (commenting out harvest logic for now as it depended on git-tracked tasks). Amp-Thread-ID: https://ampcode.com/threads/T-ac41b9b6-d117-46de-9e4f-842887a22f1d Co-authored-by: Amp <amp@ampcode.com>
2025-11-24task: migrate storage from jsonl to sqliteBen Sima
Removes .tasks/tasks.jsonl and replaces it with a local SQLite database (.tasks/tasks.db). Adds --db flag to CLI. Removes sync command. Amp-Thread-ID: https://ampcode.com/threads/T-ac41b9b6-d117-46de-9e4f-842887a22f1d Co-authored-by: Amp <amp@ampcode.com>
2025-11-24Remove git actions from workerBen Sima
2025-11-24fix(worker): remove unnecessary reset to worker branchBen Sima
2025-11-24Allow worker to take a specific task to work onBen Sima
2025-11-24feat: implement t-1o2egbj8o0n.1Ben Sima
2025-11-24feat(jr): Create Omni/Jr.hs main entry pointBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-663b6704-a8b0-4983-a62f-0ef00c61410c Co-authored-by: Amp <amp@ampcode.com>
2025-11-24Display credits correctly and don't loop agentBen Sima
2025-11-24Simplify agent commandBen Sima
I think the cd'ing and stuff was messing with the direnv assumptions.
2025-11-24Restore AGENTS.md instructions to workerBen Sima
2025-11-22Merge branch 'task/t-1o2cbco62ly' into liveBen Sima
2025-11-22feat(agent): restore vertical status layoutBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-cb6b70cf-bfac-4ef2-bad9-280aa47efacf Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-1o2cbco62lyBen Sima
2025-11-22task: complete t-1o2c9vazf64 (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-1o2c9vazf64Ben Sima
2025-11-22fix: cleanup test artifact in Omni/Task/RaceTest.hsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22fix: remove redundant imports in Omni/Agent/Log.hsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22fix: fix compilation and lint errorsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22fix: fix compilation errors in Omni/Ci.hsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22Merge branch 'review/t-1o2bxd3kezj' into liveBen Sima
2025-11-22task: complete t-1o2bxd11zv9 (Merge)Ben Sima
https: //ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com> Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221
2025-11-22Merge branch 'review/t-1o2bxcq7999.4' into liveBen Sima
2025-11-22chore: remove legacy Omni/Ci.shBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22Merge branch 'review/t-1o2bxcq7999.3' into liveBen Sima
2025-11-22task: complete t-1o2bxcq7999.2 (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-1o2bxd3kezjBen Sima
The task was to update `Omni/Task/Core.hs` to exclude tasks with `taskType == Epic` from the `task ready` command. I have: 1. Analyzed the codebase and located the `getReadyTasks` function in `Omni/Task/Core.hs`. 2. Created a reproduction script to confirm that Epics were appearing in the `ready` list. 3. Modified `Omni/Task/Core.hs` to add `taskType task /= Epic` condition in `getReadyTasks`. 4. Added a unit test in `Omni/Task.hs` to verify that Epics are excluded from ready tasks. 5. Ran tests using `bild --test Omni/Task.hs` and verified they pass. 6. Ran `lint Omni/Task.hs Omni/Task/Core.hs` to ensure code quality. The changes are verified and ready. ```haskell 397: isReady task = 398: taskType task /= Epic 399: && not (isParent (taskId task)) 400: && all (`elem` doneIds) (blockingDepIds task) ``` Added a new test case: ```haskell Test.unit "ready tasks exclude epics" <| do epic <- createTask "Epic task" Epic Nothing Nothing P2 [] Nothing ready <- getReadyTasks (taskId epic `notElem` map taskId ready) Test.@?= True, ```
2025-11-22feat: implement t-1o2bxcq7999.4Ben Sima
I have completed the task. 1. **Analysis**: I located `Omni/Agent/start-worker.sh` and identified the correct location to insert the `git sync` command (before building `task` and `agent`). 2. **Implementation**: I modified `Omni/Agent/start-worker.sh` to run `git sync` inside the worker directory. 3. **Verification**: * Ran `lint Omni/Agent/start-worker.sh` (passed). * Ran `bash -n Omni/Agent/start-worker.sh` to check syntax (passed). * Ran `bild --test Omni/Agent.hs` to ensure no regressions in the associated Haskell code (passed). The `start-worker.sh` script now syncs the worker repository before building the necessary tools, ensuring the worker runs with the latest code. Files updated: - `Omni/Agent/start-worker.sh`
2025-11-22feat: implement t-1o2bxcq7999.3Ben Sima
2025-11-22task: complete t-1o2bxcq7999.1 (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-1o2bxd11zv9Ben Sima
The task to fix missing Time, Thread, and Credits in the Agent Log has been completed. **Changes Implemented:** 1. **`Omni/Agent/Log.hs`**: * Added `Data.Aeson` and `Data.ByteString` imports for JSON parsing. * Updated `Status` data type to include `statusThread`. * Implemented `LogEntry` data type and `FromJSON` instance to match the `amp` log format. * Added `processLogLine` function to parse JSON log lines and update the global status. * Updated `render` function to display the Thread ID. * Added logic to extract and format `Time` and `Credits` from log entries. 2. **`Omni/Agent/Worker.hs`**: * Added a log monitoring thread using `forkIO` in `runAmp`. * Implemented `monitorLog` to tail the `_/llm/amp.log` file and pass lines to `AgentLog.processLogLine`. * Added `waitForFile` to ensure the log monitor waits for the log file to be created. **Verification:** * Verified that both `Omni/Agent/Log.hs` and `Omni/Agent/Worker.hs` compile successfully using `bild` (ignoring the expected "no main" error for library modules). * Ran `lint` on both files with no errors. The agent status bar should now correctly display the Thread ID, elapsed/current Time, and Credits usage as parsed from the `amp` logs.
2025-11-22feat: implement t-1o2bxcq7999.2Ben Sima
The task "Add Approved status to Omni/Task" has been implemented. **Changes made:** 1. **`Omni/Task/Core.hs`**: * Updated `Status` enum to include `Approved`. * Updated `TaskStats` record to include `approvedTasks` count. * Updated `getTaskStats` to count `Approved` tasks. * Updated `showTaskStats` to display the count. * Updated `printTreeNode'` and `printTask` to visualize `Approved` status with `[+]` symbol and green color. 2. **`Omni/Task.hs`**: * Updated `help` documentation to list `approved` as a valid status. * Updated `list` command to support filtering by `--status=approved`. * Updated `update` command to support setting status to `approved`. * Added unit tests for the new CLI functionality. **Verification:** * Ran `bild --test Omni/Task.hs` (with `CODEROOT` explicitly set to current directory to bypass build caching issue) and all tests passed. * Manually verified creating a task, updating it to `approved`, showing it, viewing the tree, and viewing stats using the compiled binary. **Note on Build Environment:** * I encountered an issue where `bild` would not rebuild because `CODEROOT` was pointing to `/home/ben/omni` instead of the worker workspace `/home/ben/omni-worker-3`. I temporarily set `export CODEROOT=$(pwd)` to successfully build and verify the changes. The `Approved` status is now fully supported in the Task core and CLI, enabling the review workflow described in the plan.
2025-11-22feat: implement t-1o2bxcq7999.1Ben Sima
2025-11-22task: complete t-1o2bkufixnc (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22fix: remove unused import in Omni/Agent.hsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22fix: fix compilation errors in Omni/Agent.hsBen Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-1o2bkufixncBen Sima
Refactored `AGENTS.md` to be a concise cheat sheet by: 1. Moving detailed examples of "Forbidden Patterns" and "Bug Discovery" to `Omni/Task/README.md`. 2. Moving "Coding Conventions" (Always include tests, No TODOs, Fast typechecking) to `README.md`. 3. Removing "About Omnirepo" and "Source Layout" sections from `AGENTS.md` as they are duplicative of `README.md`. 4. Condensing `AGENTS.md` to contain only Critical Rules, a short Session Checklist/Cheat Sheet, and pointers to the detailed READMEs (`Omni/Task/README.md`, `Omni/Bild/README.md`, `Omni/Ide/README.md`, `README.md`). Verified that detailed documentation exists in the respective namespaces and passed linting.
2025-11-22task: complete t-1o2bkseag8u (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22Merge branch 'review/t-rWcqsDZFM.3' into liveBen Sima
2025-11-22task: complete t-rWcqsDZFM.2 (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-rWcqsDZFM.3Ben Sima
Consolidated `monitor.sh` and `monitor-worker.sh` into a single `monitor.sh` script. 1. Updated `Omni/Agent/monitor.sh`: - Default behavior now uses `jq` to filter logs (formerly `monitor-worker.sh` behavior). - Added `--raw` flag to support raw log tailing (original `monitor.sh` behavior). - Accepts worker name as an argument (e.g., `./monitor.sh --raw omni-worker-2`). 2. Deleted `Omni/Agent/monitor-worker.sh`. 3. Updated `Omni/Agent/DESIGN.md` to reference the consolidated script. 4. Verified syntax of the new script. 5. Ran tests for `Omni/Agent.hs` (passed). The new usage for `monitor.sh` is: ```bash ./Omni/Agent/monitor.sh [worker-name] # Formatted output (default) ./Omni/Agent/monitor.sh --raw [worker-name] # Raw output ```
2025-11-22task: complete t-rWcqsDZFM.1 (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22task: complete t-rWcpygi7d (Merge)Ben Sima
Amp-Thread-ID: https://ampcode.com/threads/T-ca3b086b-5a85-422a-b13d-256784c04221 Co-authored-by: Amp <amp@ampcode.com>
2025-11-22feat: implement t-1o2bkufixncBen Sima
I have successfully refactored `AGENTS.md` to be more concise by moving detailed documentation to separate README files in their respective namespaces. **Changes made:** 1. **Created `Omni/Task/README.md`**: Contains the detailed "Task Manager for AI Agents" documentation. 2. **Created `Omni/Bild/README.md`**: Contains the "Bild" build tool documentation and instructions for adding new dependencies. 3. **Created `Omni/Ide/README.md`**: Contains the "Development Tools" (run.sh, lint, etc.) and "Git Workflow" documentation. 4. **Updated `AGENTS.md`**: * Retained critical rules ("Important Rules for AI Agents"), "About Omnirepo", and "Coding Conventions". * Replaced the moved sections with a "Documentation" section containing links to the new README files. * Used relative paths for links to ensure portability. **Verification:** * Ran `bild --test Omni/Task.hs` and `bild --test Omni/Bild.hs` to ensure no regressions were introduced in the touched namespaces. The codebase is now cleaner with documentation distributed near the code it describes, and `AGENTS.md` is now a focused entry point for agents.
2025-11-22feat: implement t-1o2bkseag8uBen Sima
The task to automate the git commit template configuration has been completed. **Changes Implemented:** 1. **Created `.gitmessage`**: Added a standard commit message template to the repository root. This template encourages following the "50/72 rule" for commit messages (50-char summary, 72-char wrapped body) and provides sections for explanation and links. 2. **Updated `Omni/Task.hs`**: Modified the `init` command handler in the task manager CLI. Now, when `task init` is run, it executes `git config commit.template .gitmessage` in addition to initializing the task database. **Verification:** - Ran `bild --test Omni/Task.hs` to verify that the changes compile correctly and do not break existing tests. - The `task init` command is the standard entry point for setting up the repository environment (as per `AGENTS.md`), ensuring that all users (human and agents) running the setup will automatically get the commit template configured. **Files Created/Modified:** - `Omni/Task.hs` (modified) - `.gitmessage` (created) The changes are ready for the system to commit. [Omni/Task.hs](file:///home/ben/omni-worker-3/Omni/Task.hs) [.gitmessage](file:///home/ben/omni-worker-3/.gitmessage)
generated by cgit v1.2.3 (git 2.25.1) at 2025-12-16 15:17:44 +0000