Skip to content

fix: secure setup service env file - #42

Merged
Kingson4Wu merged 1 commit into
devfrom
loop/english-pilot/bug-fix/1789738200000-english-pilot-bug-fix
Sep 18, 2026
Merged

Kingson4Wu merged 1 commit into
devfrom
loop/english-pilot/bug-fix/1789738200000-english-pilot-bug-fix

Conversation

@Kingson4Wu

Copy link
Copy Markdown
Contributor

Summary

  • create and update the setup service .env with mode 0600
  • preserve non-writing setup planning so english-pilot setup --json does not create runtime directories or .env
  • add regression coverage for both permission hardening and no-mutation planning

Verification

  • npm run typecheck
  • npm test
  • pnpm exec vitest run tests/integration/installer.test.ts
  • loop architecture assessment: score 98 / target 95
  • push hook project-health suite passed during branch push

Notes

  • One confirmed production-risk permissions bug fixed. Deferred non-confirmed candidates: old session normalization and setup wording.

@Kingson4Wu
Kingson4Wu merged commit 4d6ba49 into dev Sep 18, 2026
7 checks passed
@Kingson4Wu
Kingson4Wu deleted the loop/english-pilot/bug-fix/1789738200000-english-pilot-bug-fix branch September 18, 2026 14:55
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