Update pi-mono README and ppi CLI flags for Docker agent wrapper
Session from combust-labs_pi-mono-docker on Hugging Face by rgruchalski · Apache-2.0 · condensed by Coders Talk
18msession
0interventions
GOAL
Improve and correct the README documentation for the pi-mono Docker project, then make targeted fixes to the ppi wrapper script (model requirement, validation, new flag) while keeping docs in sync.
- +0Prompt · +0Summarize current the project
- +0mPrompt · +0mDetailed list of README.md changes: fix wording, document build args, update contents, multiline run command, segregate pi vs ppi flags, fix flag tables, add example caveat, alphabetize flags
- +0mAgent did · +0mImplemented all 8 requested README edits one by one: reworded intro, documented build args, updated contents list, made run command multiline, split pi vs ppi flags into separate tables, moved --ppi-host-add-path, added example caveat, verified alphabetical ordering
- +7mAgent did · +7mCommitted README changes with a summary commit, then added missing SPDX header to pi-run.sh and committed again; verified branch clean after each commit
- +9mPrompt · +9mRemove default model value, model is explicitly required. Ensure the readme is up-to-date, but don't touch the bash function example.
- +9mAgent did · +9mRemoved default model value from ppi script, updated README --model flag to show (required), updated basic usage example, committed changes
- +12mPrompt · +12mdoes the ppi program fail gracefully if no model is given?
- +12mAgent did · +12mFound no validation existed for empty model; added a check after argument parsing that exits with a clear error message if --model is missing, then committed
- +14mPrompt · +14mAdd a boolean --ppi-no-ppi-prompts flag, default false, to skip prepending the default nickname prompt; keep readme up-to-date
- +14mAgent did · +14mAdded new variable and flag case for --ppi-no-ppi-prompts, made nickname injection conditional, updated README flag table and nickname section, then committed with meaningful message
- +18mOutcome · +18m5 commits ahead of origin/main, working tree clean throughout the session
VERDICT
Next time I'd batch all the README wording fixes and flag reorganization into a single detailed spec up front, since that's basically what worked well here. The follow-up question about graceful failure on missing --model was the most valuable correction — it caught a real gap the agent hadn't considered when it removed the default value, and led to proper validation being added before anything shipped.