Skip to content

tui: write .env, .gitignore and secrets/ next to the compose file - #4

Merged
casablanque-code merged 1 commit into
mainfrom
sprint/roadmap-v0.5
Oct 14, 2026
Merged

casablanque-code merged 1 commit into
mainfrom
sprint/roadmap-v0.5

Conversation

@casablanque-code

Copy link
Copy Markdown
Owner

Secret conversion used to write .env, .gitignore and secrets/ into the shell's working directory. Compose resolves .env (project directory) and the file: paths of top-level secrets relative to the compose file, so opening ~/proj/compose.yaml from another directory scattered those files in the wrong place and left the stack broken.

The model now remembers the directory of the compose file it is bound to (set on import and on successful save) and convertSecretCmd writes everything there. A session not yet bound to a file keeps the old behavior (working directory).

Tests cover both strategies, the unbound fallback, and the binding on import/save.

What changed

How to test

Checklist

  • make lint / npm run lint passes
  • make test / npm test passes
  • Documentation updated (if needed)
  • Commits follow Conventional Commits

Secret conversion used to write .env, .gitignore and secrets/ into the
shell's working directory. Compose resolves .env (project directory)
and the file: paths of top-level secrets relative to the compose file,
so opening ~/proj/compose.yaml from another directory scattered those
files in the wrong place and left the stack broken.

The model now remembers the directory of the compose file it is bound
to (set on import and on successful save) and convertSecretCmd writes
everything there. A session not yet bound to a file keeps the old
behavior (working directory).

Tests cover both strategies, the unbound fallback, and the binding on
import/save.
@casablanque-code
casablanque-code merged commit fc4e085 into main Oct 14, 2026
4 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.

1 participant