Skip to content

fix(#298): add install guidance note to bundled notion-cli plugin - #379

Merged
javimosch merged 1 commit into
masterfrom
am/am-f17c27-dknmen3vcxcb-0b34c3c0
Aug 13, 2026
Merged

fix(#298): add install guidance note to bundled notion-cli plugin#379
javimosch merged 1 commit into
masterfrom
am/am-f17c27-dknmen3vcxcb-0b34c3c0

Conversation

@javimosch

@javimosch javimosch commented Aug 13, 2026

Copy link
Copy Markdown
Owner

Automated maintenance run by automaintainer.

Focus: == ASSIGNED OBJECTIVE ==
Fix GitHub issue #298 ONLY: Add 4ier/notion-cli as a bundled plugin in SuperCLI. PR title MUST reference #298.


OPEN PR AWARENESS (secondary — do not replace the ASSIGNED OBJECTIVE):
These open pull requests are already open and awaiting review. Do NOT start UNRELATED work on the files they touch. If your ASSIGNED OBJECTIVE requires editing one of those files, complete the objective anyway. Never abandon the objective to pick a different GitHub issue just to avoid overlap.

Branch: am/am-f17c27-dknmen3vcxcb-0b34c3c0

Diff:

plugins/notion-cli/plugin.json | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Summary by CodeRabbit

  • Documentation
    • Added installation guidance covering binary naming and linking requirements.
    • Documented alternative installation methods and token storage location.
    • Clarified behavior when piping JSON output.

The bundled notion-cli plugin already had a detailed note in its
install-guidance.json, but the manifest's install_guidance did not
include it. This meant `sc plugins install` and `sc plugins doctor`
showed an empty note for the plugin. Add the note to plugin.json so
install and doctor output explain the binary rename step, alternative
install methods (Homebrew, npm, GitHub Releases, Scoop), token storage,
and JSON auto-detection.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 7b9a8dd3-3e4f-4a90-bbad-c826648732c5

📥 Commits

Reviewing files that changed from the base of the PR and between 981d694 and 131e0c0.

📒 Files selected for processing (1)
  • plugins/notion-cli/plugin.json

📝 Walkthrough

Walkthrough

The Notion CLI plugin updates its installation guidance with binary naming, installation alternatives, token storage, and piped JSON output details.

Changes

Notion CLI installation guidance

Layer / File(s) Summary
Update installation guidance
plugins/notion-cli/plugin.json
The install_guidance note documents alternative installation methods, the notion-cli to notion binary link requirement, token configuration, and piped JSON output.

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

Mergeability Score: ⚪ Minimal · up to 131e0

This localized plugin guidance update introduces no actionable merge-blocking risk; it is merge-ready after normal checks and review.

Possibly related PRs

🚥 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 identifies the fix and the added installation guidance for the bundled notion-cli plugin.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch am/am-f17c27-dknmen3vcxcb-0b34c3c0

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

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

@javimosch
javimosch merged commit 319b705 into master Aug 13, 2026
0 of 2 checks passed
@javimosch
javimosch deleted the am/am-f17c27-dknmen3vcxcb-0b34c3c0 branch August 13, 2026 07:50
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