docs: sharpen LIMITATIONS boundaries (tool path + key custody)#161
Closed
sergeyenin wants to merge 1 commit into
Closed
docs: sharpen LIMITATIONS boundaries (tool path + key custody)#161sergeyenin wants to merge 1 commit into
sergeyenin wants to merge 1 commit into
Conversation
Fold two honest scope points from the community PR #155 (@ded-furby) into the existing LIMITATIONS.md: - Tool-governance: note that Talon does not prevent the same tool from being invoked on a path that does not pass through Talon. - Evidence: state the signing-key-custody assumption inline in the integrity claim ("assuming that key remains protected"). Co-authored-by: ded-furby <ded-furby@users.noreply.github.com>
Contributor
Author
|
Closing this in favor of giving @ded-furby ownership of the improvement in #155 — the two refinements here (the "same tool invoked on a path outside Talon" scope note and folding the signing-key-custody assumption into the HMAC claim) are better landed by the original author. See my note on #155. Branch is preserved in case we need to revive it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Enhances the existing
LIMITATIONS.mdwith two honest scope points contributed in the community PR #155 by @ded-furby, adapted into the current structure:Context
LIMITATIONS.mdalready landed onmain(closing #117) with a capability status table, source-code links, EU-routing/PII honesty, and docs-surface cross-links. PR #155 proposed a parallel, standalone version that now conflicts withmainand no longer closes #117. Rather than lose its best ideas, this PR captures the two unique, high-value lines from #155 and credits the author viaCo-authored-by.Test plan
scripts/check-claim-discipline.sh).Note
Low Risk
Documentation-only edits to LIMITATIONS.md with no runtime or security behavior changes.
Overview
LIMITATIONS.mdtightens two scope statements so reviewers see where Talon’s claims stop.The evidence boundary now states that a valid signature means the record was signed with the deployment key and, assuming that key stays protected, was not altered after signing—making key custody an explicit assumption in the integrity claim.
The tool-governance boundary adds that Talon does not block the same tool from running on traffic that bypasses Talon (only the gateway path gets request-body filtering today).
Reviewed by Cursor Bugbot for commit c9a8d59. Configure here.