Add start flags, skip-install optimization, and pier patch-harbor - #1
Open
mikerjacobi wants to merge 8 commits into
Open
Add start flags, skip-install optimization, and pier patch-harbor#1mikerjacobi wants to merge 8 commits into
mikerjacobi wants to merge 8 commits into
Conversation
mikerjacobi
changed the title
jbragg
approved these changes
Mar 19, 2026
jbragg
left a comment
jbragg
left a comment
Collaborator
There was a problem hiding this comment.
see readme suggestions & ci test failure
| - `--agent` installs a coding agent. Can be called multiple times to install multiple agents. When `task_path` is omitted, operates on the current workspace. | ||
| - `-f` / `--force` tears down any existing workspace or container at the target directory before starting fresh. | ||
| - `--ae` / `--agent-env` passes an environment variable to the agent inside the container (`KEY=VALUE`). Repeatable. These are persisted in the session and forwarded by `pier exec`. | ||
| - `--ee` / `--environment-env` passes an environment variable into the container environment at startup (`KEY=VALUE`). Repeatable. |
Collaborator
There was a problem hiding this comment.
Example when to use this instead of --ae would be nice
| - `-d` specifies the workspace directory. Required for local tasks; defaults to `./<task-name>` for remote tasks. | ||
| - `--host` skips the container (workspace only). | ||
| - `--agent` installs a coding agent. Can be called multiple times to install multiple agents. When `task_path` is omitted, operates on the current workspace. | ||
| - `-f` / `--force` tears down any existing workspace or container at the target directory before starting fresh. |
Collaborator
There was a problem hiding this comment.
What do you think about --delete or --clean to be more explicit that the workspace dir will be deleted?
Made-with: Cursor
Made-with: Cursor
…s in README - --force only allows non-empty dirs; --delete tears down existing session at -d - README: when to use --ee (compose startup) vs -e/--env-file/--ae (pier exec) - Rename test to test_start_delete_tears_down_existing Made-with: Cursor
mikerjacobi
force-pushed
the
mike/various-flags
branch
from
April 7, 2026 04:11
887ce1c to
049affe
Compare
…rial Made-with: Cursor
mikerjacobi
force-pushed
the
mike/various-flags
branch
from
April 7, 2026 21:25
a50f991 to
588e7c5
Compare
… env forwarding - Remove -f/--force; non-empty workspace dirs now warn instead of error - --exec prints a message on non-zero exit before propagating the code - _start_task_free forwards agent_env/environment_env when installing agents into an already-running container Made-with: Cursor
mikerjacobi
force-pushed
the
mike/various-flags
branch
from
April 7, 2026 22:34
26d89bf to
58805ee
Compare
Made-with: Cursor
- Add `pier/patches/` package with two idempotent harbor patches: - trial-chown: fix agent log permissions for trajectory extraction - skip-install: skip agent install when binary is already in the image - Add `pier patch-harbor` CLI command that discovers and patches harbor across all uv tool venvs (both standalone harbor and pier's copy) - Add `_is_agent_installed()` probe so pier start skips redundant agent setup when the binary is already present in the container (~5min saved) - Add `skip_install` flag to `setup_agent()` / harbor_bridge to run only interactive setup (onboarding, skills, MCP) without the install step Made-with: Cursor
mikerjacobi
changed the title
Made-with: Cursor
jbragg
force-pushed
the
main
branch
2 times, most recently
from
July 8, 2026 06:01
516b19e to
27ef61b
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
--ae/--agent-envand--ee/--environment-envonpier start: agent/session env (persisted, forwarded onpier exec) vs compose service env at container start.--delete: stop container, remove workspace tree and session, then start fresh.--execonpier start: run a command in the container immediately after start.--allonpier stop: stop all active container-mode workspaces.pier patch-harbor: new CLI command that applies idempotent patches to harbor's installed source across all uv tool venvs:Test plan
olmoearth-harbor.shdebug/togo — reward 0.709, 0 errorsshippy-harbor.shwhat-is-skylight — reward 0.300, 0 errorsolmoearth-pier.shdebug/togo — completed in ~2 min, 0 errorsshippy-pier.shwhat-is-skylight — completed in 84s, 0 errorspier patch-harboridempotent (second run reports "already applied")