docs: refresh stale AGENTD_* env vars and old binary names in AGENTS.md#397
Merged
Conversation
The path/shell/assets/session env vars were renamed to the CONSTRUCT_*
prefix, but the AGENTS.md TUI-recording and hot-reload recipes still
referenced the old AGENTD_* names — so following the recipe verbatim
silently fails to isolate the daemon (it reads the real config/state
dirs instead). Update all 11 occurrences to match the code:
AGENTD_{RUNTIME,STATE,DATA,CONFIG}_DIR -> CONSTRUCT_*_DIR
AGENTD_SHELL_BIN -> CONSTRUCT_SHELL_BIN
AGENTD_ASSETS_DIR -> CONSTRUCT_ASSETS_DIR
AGENTD_SESSION_KIND -> CONSTRUCT_SESSION_KIND
The recording/workflow recipes still referenced the pre-rename binary
names (`agent`, `agentd`, `agentd-mcp`, `agentd-adapter-*`) and the old
`/agents restart` slash command. Update them to the current names:
agent -> construct
agentd -> constructd (recipe uses `constructd run`, the
back-compat alias, so it's valid both
before and after the binary merge)
agentd-mcp -> construct-mcp
agentd-adapter-* -> construct-adapter-*
/agents restart -> /construct restart
Deliberately left unchanged: the repo/checkout directory (`~/agentd`,
`/Users/you/agentd/...`), the `/tmp/agentd-rain-demo` scratch dir, and
prose uses of "agent" that mean the concept ("agent response", "future
agents", "changing agentd") rather than the binary.
edwin-zvs
changed the title
Merged
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
1 participant
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.
What
AGENTS.md's TUI-recording, hot-reload, and workflow recipes referenced names from before the project rename, so following them verbatim silently misbehaves. Two related fixes:1. Env vars (
AGENTD_*→CONSTRUCT_*) — the code readsCONSTRUCT_*(verified: noAGENTD_*env reads remain incrates/). The stale names meant the recording recipe failed to isolate the daemon and fell back to the real config/state/data dirs.AGENTD_{RUNTIME,STATE,DATA,CONFIG}_DIR→CONSTRUCT_*_DIRAGENTD_SHELL_BIN→CONSTRUCT_SHELL_BINAGENTD_ASSETS_DIR→CONSTRUCT_ASSETS_DIRAGENTD_SESSION_KIND→CONSTRUCT_SESSION_KIND2. Binary + slash-command names (pre-rename → current):
agent→constructagentd→constructd(recipe usesconstructd run, the back-compat alias, valid before and after the binary merge in Consolidate construct + constructd into one binary #396)agentd-mcp→construct-mcpagentd-adapter-*→construct-adapter-*/agents restart→/construct restart(verified againstcrates/cli/src/app.rs)Deliberately NOT changed
~/agentd,/Users/you/agentd/.../tmp/agentd-rain-demoscratch-dir labelDoc-only; no code change. (Spotted while testing #396.)
🤖 Generated with Claude Code