Skip to content

Make /workspace the default working directory on Substrate - #259

Merged
rakyll merged 1 commit into
mainfrom
workspace
Jul 7, 2026
Merged

Make /workspace the default working directory on Substrate#259
rakyll merged 1 commit into
mainfrom
workspace

Conversation

@rakyll

Copy link
Copy Markdown
Member

Fixes#258.

@rakyll
rakyll requested a review from wjjclaudJuly 7, 2026 01:39
@wjjclaud

wjjclaud commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Shall we also update the SKILLS_DIR to /workspace/skills? Otherwise LGTM.

@rakyll

Copy link
Copy Markdown
MemberAuthor

No, skills shouldn't be in the working directory. Skills shouldn't be updated by the harness itself. They typically are ~/.agents/skills on development machines and are intentionally not in the working directory of the coding agent.

@rakyll
rakyll merged commit 108c5e3 into mainJul 7, 2026
8 checks passed
@rakyll
rakyll deleted the workspace branch July 7, 2026 02:43
rakyll pushed a commit that referenced this pull request Jul 21, 2026
This PR moves the internal_only boolean from `AgentOutputs` to
`Message`.
1. Internal-only messages are persisted in event log as `Message`. The
`internal_only` bit is not persisted.
2. I'd like to store some execution metadata as an internal event, e.g.
conversation id, task id, etc.
3. When resume, these internal metadata is read by AX controller, but I
want to filter them out of the history that we feed into planner agents
or custom agents since these messages are of no meaning to them.
Thus, moving it to Message level for log filtering.
Sign up for freeto 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.

Make /workspace the default working directory on Substrate

2 participants

@rakyll@wjjclaud