Skip to content

docs: say that a snapshotter can tag environments - #457

Merged
sami-alajrami merged 1 commit into
mainfrom
claude/7416-snapshotter-tag-environments
Oct 15, 2026
Merged

sami-alajrami merged 1 commit into
mainfrom
claude/7416-snapshotter-tag-environments

Conversation

@sami-alajrami

Copy link
Copy Markdown
Contributor

Say on the role pages that a snapshotter can set and remove tags on an environment, which
the server allows from kosli-dev/server#7418. The permissions matrix now has a row for
tagging environments, open to snapshotters, and one for flows, repos and controls, which
stays with members and above.

Part of kosli-dev/server#7416

🤖 Generated with Claude Code

The server now lets a snapshotter API key set and remove tags on an
environment (kosli-dev/server#7416), so the role pages said less than
the role can do. The permissions matrix splits tagging into a row for
environments, open to snapshotters, and one for flows, repos and
controls, which stays with members and above. The Snapshotter section,
the overview table, the API footnote and the two one-line role
summaries say the same.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@mintlify

mintlify Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated
kosli 🟢 Ready View Preview Oct 14, 2026, 1:22 PM

@sami-alajrami

Copy link
Copy Markdown
Contributor Author

Merge order

Merge after kosli-dev/server#7418 is deployed, or the docs describe a permission a snapshotter key does not yet have.

Pages touched

  • administration/managing_users/roles_in_kosli.md: overview table, permissions matrix, the API footnote, and the Snapshotter section (a Tags line, and the "cannot change an environment" line now excepts tags).
  • administration/authentication/service_accounts.md and implementation_guide/phase_1/roles_and_responsibilities/overview.md: the one-line role summaries.

@claude

claude Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Claude finished @sami-alajrami's task in 36s —— View job


Doc review

  • Read the diff and each changed file at the branch head
  • Checked the change against the rest of the site (snapshotter mentions, the kosli tag reference)
  • Posted the review

No findings cleared the bar. The snapshotter tagging change is applied consistently in all three changed files, and no other page on the site contradicts it. The full review is in this comment.

@claude

claude Bot commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

Doc review

No findings cleared the bar. The tagging change is applied consistently across all three role pages: the summary table, the permissions matrix, the † footnote and the Snapshotter accordion on roles_in_kosli.md, plus the role lists in service_accounts.md and the implementation guide overview. kosli tag environment is a valid form according to client_reference/kosli_tag.md, and no other page on the site describes what a snapshotter can do in a way that this change contradicts. The PR adds no links.

0 Critical, 0 Improvement, 0 Suggestion. This review is advisory only and does not block the merge.

@sami-alajrami sami-alajrami changed the title Say that a snapshotter can tag environments docs: say that a snapshotter can tag environments Oct 14, 2026
@sami-alajrami
sami-alajrami marked this pull request as ready for review October 15, 2026 06:46
@sami-alajrami
sami-alajrami requested a review from a team as a code owner October 15, 2026 06:46
@sami-alajrami
sami-alajrami enabled auto-merge (squash) October 15, 2026 06:46
@sami-alajrami
sami-alajrami merged commit 75883fa into main Oct 15, 2026
9 of 10 checks passed
@sami-alajrami
sami-alajrami deleted the claude/7416-snapshotter-tag-environments branch October 15, 2026 06:59

This branch was successfully deployed

1 active deployment
staging — a8f87101 Deployed Oct 14, 2026 by mintlify[bot]
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