[thoroc]

Just noticed that kingfisher wasn't included in the builtins - this PR addresses that.

Summary by CodeRabbit

• New Features
  • Added support for installing MongoDB’s Kingfisher tool through mise.
  • Pinned Kingfisher to version 1.109.0 for consistent setup.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Pro Plus

Run ID: d52f1572-7740-4c61-97da-1376cf548fa2

📥 Commits

Reviewing files that changed from the base of the PR and between dc35871 and 78c4d97.

⛔ Files ignored due to path filters (1)

• pkl/builtins/kingfisher.pkl is excluded by !**/*.pkl

📒 Files selected for processing (1)

• test/builtin_tool_stubs/kingfisher

---

📝 Walkthrough

Walkthrough

The PR adds an executable mise tool stub for github:mongodb/kingfisher and pins it to version 1.109.0.

Changes

Kingfisher tool stub

Layer / File(s)	Summary
Add pinned Kingfisher stub test/builtin_tool_stubs/kingfisher	The test stub declares github:mongodb/kingfisher at version 1.109.0 and marks it executable.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Poem

A rabbit found a tool stub bright,
Pinned Kingfisher just right.
1.109.0 in place,
Ready for the test-run race.
Hop, hop—clean and light!

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes adding Kingfisher as a built-in secret scanner.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[greptile-apps]

Greptile Summary

Adds a Kingfisher secret-scanning builtin, including pass/fail fixtures and a pinned executable stub. The generated Pkl catalogue was not updated, so the new builtin is not yet exposed through Builtins.kingfisher.

• Defines Kingfisher as a text-file check with update checks and live validation disabled.
• Tests clean content and a synthetic Slack token with Kingfisher's findings exit code.
• Adds a version 1.109.0 tool stub for builtin tests.

Confidence Score: 4/5

The PR should not merge until the tracked Pkl builtin catalogue is regenerated so Builtins.kingfisher is available and the render check remains clean.

The scanner definition and test stub are internally consistent, but adding the source file alone leaves the generated public Pkl export stale and causes both user configuration resolution and the generated-file CI check to fail.

Files Needing Attention: pkl/builtins/kingfisher.pkl and pkl/Builtins.pkl

Important Files Changed

Filename	Overview
pkl/builtins/kingfisher.pkl	Adds the scanner definition and embedded tests, but omits regeneration of the tracked public builtin catalogue.
test/builtin_tool_stubs/kingfisher	Adds a convention-compatible executable stub pinned to Kingfisher 1.109.0.

_{Reviews (1): Last reviewed commit: "feat(builtins): add kingfisher secret sc..." | Re-trigger Greptile}