Skip to content

[codex] fix pi home archive restore - #127

Merged
rawkintrevo merged 1 commit into
mainfrom
codex/fix-pi-home-archive-restore
Jun 24, 2026
Merged

[codex] fix pi home archive restore#127
rawkintrevo merged 1 commit into
mainfrom
codex/fix-pi-home-archive-restore

Conversation

@rawkintrevo

@rawkintrevo rawkintrevo commented Jun 24, 2026

Copy link
Copy Markdown
Owner

Summary

  • exclude Pi runtime-managed $PI_CODING_AGENT_DIR/npm/node_modules from the persisted $HOME archive target
  • cover the home archive exclusion behavior with runner unit tests
  • document the archive exclusion in the runtime container wiki

Root Cause

pi-basic and related Pi images bake packages such as pi-mcp-adapter into the Pi agent npm tree. Startup then restored the persisted /root archive over that tree before Express started listening. GNU tar could fail on .pi/agent/npm/node_modules/.bin, causing Cloud Run to report that the container never listened on PORT=8080.

Validation

  • npm --prefix session-runner test
  • npm --prefix session-runner run lint
  • npm run docs:check

Browser QA intentionally skipped at user request; user will QA.

Deployment

  • FIREBASE_CLI_DISABLE_UPDATE_NOTIFIER=true firebase deploy --only functions --project pi-agents-cloud succeeded. Updated api and reapIdleSessions.
  • gcloud builds submit session-runner --project pi-agents-cloud --config session-runner/cloudbuild.pi-basic.yaml succeeded. Pushed session-runner:pi-basic digest sha256:5e65f13ed1d2195002391c27c539cfe34b54bef9294e47526ca70656300e7ce0.

Closes #126

@github-actions

Copy link
Copy Markdown

Visit the preview URL for this PR (updated for commit fa5af94):

https://pi-agents-cloud--pr127-codex-fix-pi-home-ar-5rq2h0nj.web.app

(expires Wed, 08 Jul 2026 01:58:20 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: ad3a3866d5dcd316f5eca68b23f0ce7295063677

@rawkintrevo
rawkintrevo marked this pull request as ready for review June 24, 2026 02:06
@rawkintrevo
rawkintrevo merged commit 98f6b16 into main Jun 24, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pi-basic sessions fail startup restoring Pi home archive

1 participant