Skip to content

Fix vault docs: env var name and scope format#126

Open
Aitem wants to merge 1 commit intomasterfrom
fix/6938-vault-docs-env-var-and-scope-format
Open

Fix vault docs: env var name and scope format#126
Aitem wants to merge 1 commit intomasterfrom
fix/6938-vault-docs-env-var-and-scope-format

Conversation

@Aitem
Copy link
Member

@Aitem Aitem commented Feb 25, 2026

Summary

  • Rename AIDBOX_VAULT_CONFIGBOX_VAULT_CONFIG in all vault documentation pages (code was renamed in commit 3b33678e84 of sansara#6861)
  • Fix scope format from array ["Client/id"] to object {"resource_type": "Client", "id": "id"} to match actual code implementation
  • Update scope field description in all three docs pages
  • Add BOX_VAULT_CONFIG to reference/all-settings.md (was missing)

Files changed

  • docs/configuration/secret-files.md — env var + scope format + description
  • docs/reference/all-settings.md — add BOX_VAULT_CONFIG setting
  • docs/tutorials/other-tutorials/azure-key-vault-external-secrets.md — env var + scope format + description
  • docs/tutorials/other-tutorials/hashicorp-vault-external-secrets.md — env var + scope format + description

Test plan

  • Verified env var name matches code (BOX_VAULT_CONFIG in vault_config.clj:69)
  • Verified scope format matches code spec (s/keys :req-un [::resource_type] :opt-un [::id])
  • QA tested vault integration on edge:1d5095846f — all 15 tests passed

Fixes HealthSamurai/sansara#6938
Found during QA testing of HealthSamurai/sansara#6861.

🤖 Generated with Claude Code

- Rename env var from AIDBOX_VAULT_CONFIG to BOX_VAULT_CONFIG in all docs
  (secret-files.md, azure tutorial, hashicorp tutorial)
- Fix scope format: change from array ["Client/id"] to object
  {"resource_type": "Client", "id": "id"} to match actual code
- Update scope field description to match object format
- Add BOX_VAULT_CONFIG to reference/all-settings.md

Fixes #6938

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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