Add include-unused-deps input#303
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (4)
WalkthroughThis PR adds a new optional 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
lgtm! |
The report JSON is interpolated into the run script as raw text, so a single quote in dependency copyright text (e.g. "Georg Reinke (<...>)") broke the echo '...' quoting and failed with a bash syntax error / exit 2. Pass the report via an env var instead. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
The env-var approach from the previous commit fixed the quoting bug but hit MAX_ARG_STRLEN: the attribution report is ~350KB on a single line, so placing it in an env var made the kernel reject starting bash with 'Argument list too long'. A quoted heredoc keeps the value in the script file (no argv/env size limit) and treats it literally (no quoting issues). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Summary
include-unused-depsboolean input that forwards--include-unused-depstofossa analyze, matching the corresponding flag in fossa-cli.fossa analyze(notfossa test,fossa report, or container scans).dist/.Test plan
include-unused-deps: truecauses the action to pass--include-unused-depstofossa analyzefalse) is a no-op