Skip to content

fix(cli): preserve invoked cwd for local launcher#299

Merged
tiann merged 2 commits intotiann:mainfrom
ROOOO:pr/preserve-invoked-cwd
Mar 17, 2026
Merged

fix(cli): preserve invoked cwd for local launcher#299
tiann merged 2 commits intotiann:mainfrom
ROOOO:pr/preserve-invoked-cwd

Conversation

@ROOOO
Copy link
Contributor

@ROOOO ROOOO commented Mar 17, 2026

Summary

  • preserve the original invoked working directory through nested HAPI launches instead of falling back to the CLI project cwd
  • thread that resolved cwd through session bootstrap and local launch paths across supported agents
  • add coverage for spawnHappyCLI cwd propagation

Testing

  • bun x vitest run src/utils/spawnHappyCLI.test.ts
  • bun run typecheck

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review mode: initial
No issues found in changed lines.
Residual risk: cwd propagation depends on HAPI_INVOKED_CWD; only spawnHappyCLI coverage added.
Testing: Not run (automation).
HAPI Bot

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings

  • None.

Summary
Review mode: follow-up after new commits
No issues in changed lines.
Residual risk: only spawnHappyCLI unit coverage; getInvokedCwd consumers untested.

Testing

  • Not run (automation)

HAPI Bot

@tiann tiann merged commit caa7682 into tiann:main Mar 17, 2026
2 checks passed
ROOOO added a commit to ROOOO/hapi that referenced this pull request Mar 18, 2026
@ROOOO ROOOO deleted the pr/preserve-invoked-cwd branch March 19, 2026 13:17
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.

2 participants