Skip to content

Stop auto-allowing misparsed nested interpreter peels - #673

Merged
TheGreatAxios merged 2 commits into
mainfrom
cl-6988-nested-bash
Aug 25, 2026
Merged

Stop auto-allowing misparsed nested interpreter peels#673
TheGreatAxios merged 2 commits into
mainfrom
cl-6988-nested-bash

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Collaborator

Summary

  • Marks nested bash -c / interpreter peels opaque when tokenize degrades through escaped quotes (\bash) or splits the quoted body into unsafe trailing tokens, so auto cannot accept a leaf the peeler never fully saw.
  • Caps peel depth by marking remaining nested wrappers opaque once MAX_PEEL_DEPTH is exhausted, instead of auto-allowing a truncated leaf.
  • Adds regression coverage for single-level, double-nested, escaped triple-nested, and quote-broken deep nesting redirect payloads.

Verification

  • bun run typecheck and bun run build pass (pre-commit)
  • Targeted security / authz peel behavior covered by new CL-6988 cases in classify-security.test.ts

Fixes CL-6988

@linear-code

linear-code Bot commented Aug 24, 2026

Copy link
Copy Markdown

CL-6988

@TheGreatAxios
TheGreatAxios merged commit 8a62bff into main Aug 25, 2026
5 checks passed
TheGreatAxios added a commit that referenced this pull request Aug 27, 2026
* Stop auto-allowing misparsed nested interpreter peels

* Harden nested bash auto-shell peels
TheGreatAxios added a commit that referenced this pull request Aug 27, 2026
* Stop auto-allowing misparsed nested interpreter peels

* Harden nested bash auto-shell peels
@TheGreatAxios
TheGreatAxios deleted the cl-6988-nested-bash branch August 28, 2026 00:09
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.

1 participant