Skip to content

docs(secrets): clarify age strict mode default#9737

Merged
jdx merged 3 commits into
jdx:mainfrom
risu729:docs/age-strict-default
May 9, 2026
Merged

docs(secrets): clarify age strict mode default#9737
jdx merged 3 commits into
jdx:mainfrom
risu729:docs/age-strict-default

Conversation

@risu729

@risu729 risu729 commented May 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • document that direct age encryption requires experimental features to be enabled
  • clarify that age decryption is strict by default
  • document that age.strict=false skips undecryptable values and continues resolving the environment

Testing

  • mise run docs:build

Refs #9730

risu729

This comment was marked as resolved.

@greptile-apps

greptile-apps Bot commented May 9, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR improves the age encryption documentation by adding a missing prerequisite step (enabling experimental features) to the Quick Start guide and rewriting the strict-mode section to clearly state that strict decryption is the default behavior, with explicit instructions for disabling it.

Confidence Score: 5/5

Documentation-only change with no runtime impact; safe to merge.

The change touches only a single Markdown documentation file. The new content accurately reflects the experimental gating requirement and the strict-by-default decryption behavior, correcting a previously ambiguous description. No code paths are modified.

No files require special attention.

Important Files Changed

Filename Overview
docs/environments/secrets/age.md Documentation update: adds 'enable experimental features' as step 1 in Quick Start, renumbers subsequent steps, and replaces the ambiguous single-line note about non-strict behavior with a clear explanation that strict mode is the default and how to disable it.

Reviews (3): Last reviewed commit: "docs(secrets): clarify age decryption st..." | Re-trigger Greptile

Comment thread docs/environments/secrets/age.md Outdated

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the documentation for age-encrypted secrets to include instructions for enabling experimental features and to reflect changes in decryption behavior, which is now strict by default. The review feedback suggests using the explicit set subcommand for consistency, refining the technical description of strict mode to focus on decryption, and standardizing the syntax for setting configuration values.

Comment thread docs/environments/secrets/age.md Outdated
Comment thread docs/environments/secrets/age.md Outdated
Comment thread docs/environments/secrets/age.md
@risu729

This comment was marked as outdated.

@risu729 risu729 marked this pull request as ready for review May 9, 2026 07:18
@jdx jdx merged commit 13b4516 into jdx:main May 9, 2026
34 checks passed
@risu729 risu729 deleted the docs/age-strict-default branch May 9, 2026 13:40
mise-en-dev added a commit that referenced this pull request May 10, 2026
### 🚀 Features

- add --inactive option to outdated and upgrade commands for inactive
tools by @roele in [#9640](#9640)

### 🐛 Bug Fixes

- **(aqua)** resolve bin paths for prefixed v tags by @risu729 in
[#9759](#9759)
- **(bun)** create bunx alongside bun.exe on Windows install by
@JamBalaya56562 in [#9732](#9732)
- **(dotnet)** use shared prerelease tool option by @risu729 in
[#9720](#9720)
- **(node)** use matching node in npm shim by @jdx in
[#9749](#9749)
- **(task)** resolve bash deterministically on Windows to avoid WSL
launcher by @JamBalaya56562 in
[#9750](#9750)

### 📚 Documentation

- **(secrets)** clarify age strict mode default by @risu729 in
[#9737](#9737)
- **(tasks)** add bash shebang to conditional-dependencies example by
@JamBalaya56562 in [#9747](#9747)
- update backend tool option docs by @risu729 in
[#9738](#9738)

### 📦 Registry

- remove tools with zero users by @jdx in
[#9725](#9725)
- add scalafmt
([github:scalameta/scalafmt](https://github.com/scalameta/scalafmt)) by
@pokir in [#9757](#9757)
- remove flarectl by @risu729 in
[#9756](#9756)

### Chore

- **(release)** strip pre-existing sponsor block before appending
canonical one by @jdx in [#9745](#9745)

### New Contributors

- @pokir made their first contribution in
[#9757](#9757)
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