Repository navigation
fix(skills): Gracefully load Claude skill content - #2917
Merged
Merged
Conversation
Previously, broad rejection of Claude extensions prevented otherwise usable skills from loading, including Excel documentation whose Markdown punctuation was mistaken for a dynamic shell expression. Recognize shell expression boundaries and retain unsupported dynamic content as literal text with compatibility notes. Ignore model and effort preferences in favor of the current session configuration. Keep rejecting unsupported execution and permission constraints. Surface compatibility notes during discovery and explicit loading, including remote workspaces and stable-key invocation. Update the compatibility documentation and add regression coverage. Validation: 35 runtime skill contracts and 46 Core skill tests pass. The complete officecli-xlsx skill also parses under both Claude slots with unchanged content and no compatibility warnings. Remote coverage uses a simulated filesystem; no UI or live remote validation was run.
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
1 participant
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.
Previously, broad rejection of Claude extensions prevented otherwise
usable skills from loading, including Excel documentation whose
Markdown punctuation was mistaken for a dynamic shell expression.
Recognize shell expression boundaries and retain unsupported dynamic
content as literal text with compatibility notes. Ignore model and
effort preferences in favor of the current session configuration.
Keep rejecting unsupported execution and permission constraints.
Surface compatibility notes during discovery and explicit loading,
including remote workspaces and stable-key invocation. Update the
compatibility documentation and add regression coverage.
Validation: 35 runtime skill contracts and 46 Core skill tests pass.
The complete officecli-xlsx skill also parses under both Claude slots
with unchanged content and no compatibility warnings. Remote coverage
uses a simulated filesystem; no UI or live remote validation was run.