Skip to content

docs(kits): document neutral rc snippets - #511

Merged
mogul merged 1 commit into
refactor/acq-devenv-base-imagefrom
paltry-fly
Sep 24, 2026
Merged

mogul merged 1 commit into
refactor/acq-devenv-base-imagefrom
paltry-fly

Conversation

@mogul

@mogul mogul commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

Summary

Document the neutral shell rc snippet contract for kits as part of #482.

This adds kit-author guidance for /home/agent/.rc.d/*.sh, including deterministic ordering, no-secrets guardrails, fish/nushell constraints, and migration guidance for team kits that previously owned their own shell startup loop.

AI-assisted: yes, OpenCode.

Related Issues

Refs: #482
Parent: #473

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Documentation update
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • I have read CONTRIBUTING.md
  • My commits follow the Conventional Commits format (e.g., feat:, fix:, docs:)
  • I have tested my changes locally
  • I have updated documentation as needed
  • I have not included any secrets, API keys, or sensitive information

Test Results

Verification transcript:

  • PASS: git diff --check
  • PASS: npm run lint:md
  • PASS: adversarial review sub-agent completed; one overclaim about zsh behavior was found and addressed before commit.

No Bats run was needed for this docs-only increment; existing branch tests already cover the implemented rc.d delivery and MSB bridge behavior.

Security Impact

No authentication, authorization, credential injection, or network behavior changed. The documentation explicitly reinforces that shell snippets must not contain or print secrets.

Rollback

Revert commit 58f954f.

Document the kit-owned /home/agent/.rc.d shell snippet contract, shell-specific guardrails, and team-kit migration path.

Refs: #482
@mogul
mogul requested a review from a team as a code owner September 24, 2026 04:14
@mogul
mogul merged commit a59a4a4 into refactor/acq-devenv-base-image Sep 24, 2026
3 checks passed
@mogul
mogul deleted the paltry-fly branch September 24, 2026 04:18
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