Update version numbers in post-create-cmd.sh - #49
Conversation
|
| Filename | Overview |
|---|---|
| .devcontainer/post-create-cmd.sh | Updates dependency pins and revises noninteractive installation, OmniRoute readiness, combo creation, and MCP setup. |
| .devcontainer/start-hermes.sh | Extends the Hermes dashboard readiness window from one minute to ten minutes. |
| .github/workflows/devcontainer-ci.yml | Adds workflow path filters and changes timeout and failure handling for the Mnemon integration check. |
| README.md | Updates dependency badges to match the newly pinned releases. |
Reviews (10): Last reviewed commit: "Update CI workflow to trigger on specifi..." | Re-trigger Greptile
| OMNIROUTE_VERSION=3.8.50 | ||
| MODELRELAY_VERSION=1.22.1 | ||
| MNEMON_VERSION=0.2.7 | ||
| PI_AGENT_VERSION=0.84.4 | ||
| MNEMON_VERSION=0.2.8 | ||
| PI_AGENT_VERSION=0.85.1 |
There was a problem hiding this comment.
Version references remain stale
The script now pins OmniRoute 3.8.50, Mnemon 0.2.8, and PI Agent 0.85.1, while the README and devcontainer documentation still advertise the previous versions, giving contributors inaccurate information when reproducing or diagnosing the environment.
Prompt To Fix With AI
This is a comment left during a code review.
Path: .devcontainer/post-create-cmd.sh
Line: 4-7
Comment:
**Version references remain stale**
The script now pins OmniRoute 3.8.50, Mnemon 0.2.8, and PI Agent 0.85.1, while the README and devcontainer documentation still advertise the previous versions, giving contributors inaccurate information when reproducing or diagnosing the environment.
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
There was a problem hiding this comment.
🟢 Approval recommended
The changes are limited to straightforward version pin bumps in the devcontainer setup script with no functional logic modifications.
Pull request overview
This pull request updates pinned dependency versions used by the devcontainer post-create setup script, ensuring newly created development environments install the intended latest toolchain components.
Changes:
- Bumped
OMNIROUTE_VERSIONfrom3.8.49to3.8.50 - Bumped
MNEMON_VERSIONfrom0.2.7to0.2.8 - Bumped
PI_AGENT_VERSIONfrom0.84.4to0.85.1
File summaries
| File | Description |
|---|---|
| .devcontainer/post-create-cmd.sh | Updates pinned versions for OmniRoute, Mnemon, and Pi agent installs used during devcontainer initialization |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Set CI environment variable to true to suppress npm prompts.
Increased the maximum attempts to wait for OmniRoute to be ready from 10 to 180.
…l checks to use Hermes instead of Claude
…th in devcontainer-ci.yml
This pull request updates several version numbers in the
.devcontainer/post-create-cmd.shscript to ensure the development environment uses the latest dependencies.Dependency version updates:
OMNIROUTE_VERSIONfrom3.8.49to3.8.50to pull in the latest features or fixes.MNEMON_VERSIONfrom0.2.7to0.2.8for improved functionality or bug fixes.PI_AGENT_VERSIONfrom0.84.4to0.85.1to ensure compatibility and latest improvements.