Skip to content

feat: block dangerous scope keys and harden findScope (#898)#938

Open
harttle wants to merge 2 commits into
nextfrom
feat/scope-security
Open

feat: block dangerous scope keys and harden findScope (#898)#938
harttle wants to merge 2 commits into
nextfrom
feat/scope-security

Conversation

@harttle

@harttle harttle commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Block proto, constructor, prototype in template reads
  • sanitizeScope for managed scopes
  • findScope respects ownPropertyOnly

Test plan

  • npm test context.spec scope-security

Closes #898

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@coveralls

coveralls commented Jul 15, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29426900249

Warning

No base build found for commit 61ed163 on next.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 99.554%

Details

  • Patch coverage: 18 of 18 lines across 2 files are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 2985
Covered Lines: 2976
Line Coverage: 99.7%
Relevant Branches: 1053
Covered Branches: 1044
Branch Coverage: 99.15%
Branches in Coverage %: Yes
Coverage Strength: 18938.09 hits per line

💛 - Coveralls

Co-authored-by: Cursor <cursoragent@cursor.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.

2 participants