Skip to content

Enforce core analysis input limits - #63

Merged
melonamin merged 1 commit into
masterfrom
agent/enforce-analysis-size-limits
Aug 12, 2026
Merged

Enforce core analysis input limits#63
melonamin merged 1 commit into
masterfrom
agent/enforce-analysis-size-limits

Conversation

@melonamin

Copy link
Copy Markdown
Member

Summary

  • enforce a 10 MiB UTF-8 byte limit per raw source and a 100 MiB aggregate request limit before schema, templating, or parser work
  • return structured INVALID_REQUEST issues with source attribution
  • expose the limits through the schema, generated TypeScript API, core spec, and changelog

Validation

  • controlled Codex review: no actionable findings
  • cargo test -p flowscope-core
  • cargo test -p flowscope-core --no-default-features --lib
  • schema guard and generated TypeScript checks
  • mise exec yarn@1.22.19 -- just check

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.75000% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
crates/flowscope-core/src/analyzer/input.rs 93.33% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

@melonamin
melonamin marked this pull request as ready for review August 12, 2026 16:14
@melonamin
melonamin merged commit caa9a32 into master Aug 12, 2026
9 checks passed
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