Fix default value for Action option

[grahamc]

Description

This is a rebased version of #110.

Checklist

• Tested changes against a test repository
• Added or updated relevant documentation (leave unchecked if not applicable)
• (If this PR is for a release) Updated README to point to the new tag (leave unchecked if not applicable)

Summary by CodeRabbit

• Bug Fixes
  • Corrected the default binary deny list configuration for backtrace collection to use the appropriate deny list field instead of the prefixes field.

[netlify]

✅ Deploy Preview for detsys-ts-docs ready!

Name	Link
🔨 Latest commit	b2ab394
🔍 Latest deploy log	https://app.netlify.com/projects/detsys-ts-docs/deploys/699f0fc3145635000839a4bf
😎 Deploy Preview	https://deploy-preview-144--detsys-ts-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 4c79919 and b2ab394.

⛔ Files ignored due to path filters (2)

• dist/index.mjs is excluded by !**/dist/**
• dist/index.mjs.map is excluded by !**/dist/**, !**/*.map

📒 Files selected for processing (1)

• src/index.ts

---

📝 Walkthrough

Walkthrough

The makeOptionsConfident function in src/index.ts has been modified to use actionOptions.binaryNamesDenyList as the default fallback value for binaryNamesDenyList instead of actionOptions.binaryNamePrefixes.

Changes

Cohort / File(s)	Summary
Options Configuration src/index.ts	Updated fallback logic in makeOptionsConfident to use binaryNamesDenyList field instead of binaryNamePrefixes when setting default deny list values.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~7 minutes

Poem

🐰 A tweak so neat, a switch so clean,
From prefixes old to lists pristine,
The deny list defaults find their way,
More confident options, hip-hip-hooray! 🎉

🚥 Pre-merge checks | ✅ 1 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Fix default value for Action option' is vague and overly generic, failing to specify which default value was changed or what the actual fix entails.	Consider a more descriptive title such as 'Fix default value for binaryNamesDenyList in makeOptionsConfident' to clearly indicate the specific change made.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch push-lwtrxvnsvplx

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[lucperkins]

Replaces #110

[grahamc]

Yeah I was rebasing it and for whatever reason jj lost the bookmark, and I just sighed and pushed a new branch.