fix(#298): add go install instructions to bundled notion-cli plugin by javimosch · Pull Request #372 · javimosch/supercli

javimosch · 2026-08-04T15:44:48Z

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.

PR fix(#298): add go install instructions to bundled notion-cli plugin #371 (am/am-f17c27-dkg1g7311cne-ca767030): fix(Add 4ier/notion-cli as bundled plugin #298): add go install instructions to bundled notion-cli plugin
touches: plugins/notion-cli/install-guidance.json, plugins/notion-cli/plugin.json, plugins/notion-cli/skills/quickstart/SKILL.md
PR fix(#298): add go install instructions to bundled notion-cli plugin #370 (am/am-f17c27-dkfyrq5qrxmx-e42eaa25): fix(Add 4ier/notion-cli as bundled plugin #298): add go install instructions to bundled notion-cli plugin
touches: plugins/notion-cli/install-guidance.json, plugins/notion-cli/plugin.json, plugins/notion-cli/skills/quickstart/SKILL.md
PR fix(#298): add go install instructions to bundled notion-cli plugin #369 (am/am-f17c27-dkfgj4w7jxmc-e50a95d1): fix(Add 4ier/notion-cli as bundled plugin #298): add go install instructions to bundled notion-cli plugin
touches: plugins/notion-cli/install-guidance.json, plugins/notion-cli/plugin.json, plugins/notion-cli/skills/quickstart/SKILL.md
PR fix(#335): implement run <plugin> <resource> <action> one-shot command #365 (am/am-f17c27-dkdoeotqzvn8-3120cd1a): fix(feat: sc run <plugin> <action> — one-shot discover+install+execute #335): implement run <plugin> <resource> <action> one-shot command
touches: tests/run-command.test.js, cli/help-json.js, cli/help.js, cli/run.js, cli/supercli.js
PR fix(#362): build sc-machin as static musl binary to fix GLIBC install failures #364 (am/am-f17c27-dkdjoppajdp3-a7a89ad7): fix(Cannot install #362): build sc-machin as static musl binary to fix GLIBC install failures
touches: .github/workflows/sc-machin-release.yml, README.md, supercli-machin-cli/README.md, supercli-machin-cli/install.sh

Branch: am/am-f17c27-dkg9sc8fjw7x-91e45236

Diff:

plugins/notion-cli/install-guidance.json      |  5 +++--
 plugins/notion-cli/plugin.json                |  7 ++++---
 plugins/notion-cli/skills/quickstart/SKILL.md | 17 ++++++++++++++++-
 3 files changed, 23 insertions(+), 6 deletions(-)

Summary by CodeRabbit

Documentation
- Updated Notion CLI installation guidance to prioritize Go installation with the required binary symlink.
- Added npm installation instructions.
- Clarified Homebrew, Scoop, and GitHub Releases as alternative installation methods.
- Improved version, command passthrough, and authentication guidance.

Adds the primary `go install github.com/4ier/notion-cli@latest` installation path and explains how to symlink/rename the resulting `notion-cli` binary as `notion` so the plugin's `which notion` check and all commands work. Fixes #298 Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

chatgpt-codex-connector · 2026-08-04T15:44:51Z

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

coderabbitai · 2026-08-04T15:45:21Z

📝 Walkthrough

Walkthrough

The Notion CLI guidance now uses Go installation with a notion-cli to notion symlink as the primary path. It also documents Homebrew, npm, Scoop, and GitHub Releases alternatives and updates dependency help messages.

Changes

Notion CLI installation

Layer / File(s)	Summary
Installation methods `plugins/notion-cli/install-guidance.json`, `plugins/notion-cli/plugin.json`, `plugins/notion-cli/skills/quickstart/SKILL.md`	The guidance adds `go install` and symlink setup. It retains Homebrew and adds npm, Scoop, and GitHub Releases options.
Dependency and command guidance `plugins/notion-cli/plugin.json`	Version and passthrough command messages explain the `notion-cli` binary name, symlink or rename steps, and authentication instructions.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related issues

javimosch/supercli#298: This issue concerns the bundled Notion CLI plugin and its Go-based installation.

Possibly related PRs

javimosch/supercli#367: Updates the same Notion CLI installation guidance with Go installation and symlink instructions.
javimosch/supercli#368: Modifies the same guidance files and adds Go installation details.
javimosch/supercli#369: Updates the same files with Go installation instructions and binary naming details.

🚥 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 the main change: adding Go installation instructions 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 💡 1

🛠️ Fix failing CI checks 💡

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch am/am-f17c27-dkg9sc8fjw7x-91e45236

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.}

coderabbitai

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@plugins/notion-cli/install-guidance.json`:
- Around line 6-8: Ensure the installed notion binary remains discoverable
through PATH by adding persistent PATH setup or linking it into an existing PATH
directory. Apply the same setup to plugins/notion-cli/install-guidance.json
lines 6-8, plugins/notion-cli/plugin.json lines 14-15, and
plugins/notion-cli/skills/quickstart/SKILL.md lines 61-66 before the notion
--version verification.

In `@plugins/notion-cli/plugin.json`:
- Around line 14-15: Update the installation guidance in the plugin manifest
around the notion-cli install and symlink commands to explicitly add the
resolved Go binary directory to PATH, ensuring the installed notion executable
can be found by subsequent PATH-based checks and invocations.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 67201da5-526b-481e-87a5-67567e8afc7d

📥 Commits

Reviewing files that changed from the base of the PR and between 6df5369 and 395c541.

📒 Files selected for processing (3)

plugins/notion-cli/install-guidance.json
plugins/notion-cli/plugin.json
plugins/notion-cli/skills/quickstart/SKILL.md

coderabbitai · 2026-08-04T15:51:47Z

+    "go install github.com/4ier/notion-cli@latest",
+    "GOBIN=$(go env GOBIN); GOPATH=$(go env GOPATH); ln -sf \"${GOBIN:-$GOPATH/bin}/notion-cli\" \"${GOBIN:-$GOPATH/bin}/notion\"",
    "Verify: notion --version",


🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Keep the installed notion binary on PATH.

The installation steps place the binary in the Go binary directory, but the plugin checks and launches notion through PATH.

plugins/notion-cli/install-guidance.json#L6-L8: add persistent PATH setup or link into an existing PATH directory.

plugins/notion-cli/plugin.json#L14-L15: include the same setup in manifest guidance.

plugins/notion-cli/skills/quickstart/SKILL.md#L61-L66: add the setup before notion --version.

📍 Affects 3 files

plugins/notion-cli/install-guidance.json#L6-L8 (this comment)

plugins/notion-cli/plugin.json#L14-L15

plugins/notion-cli/skills/quickstart/SKILL.md#L61-L66

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@plugins/notion-cli/install-guidance.json` around lines 6 - 8, Ensure the installed notion binary remains discoverable through PATH by adding persistent PATH setup or linking it into an existing PATH directory. Apply the same setup to plugins/notion-cli/install-guidance.json lines 6-8, plugins/notion-cli/plugin.json lines 14-15, and plugins/notion-cli/skills/quickstart/SKILL.md lines 61-66 before the notion --version verification.

coderabbitai · 2026-08-04T15:51:47Z

+      "go install github.com/4ier/notion-cli@latest",
+      "GOBIN=$(go env GOBIN); GOPATH=$(go env GOPATH); ln -sf \"${GOBIN:-$GOPATH/bin}/notion-cli\" \"${GOBIN:-$GOPATH/bin}/notion\"",


🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Keep the installed binary on PATH.

Line 15 creates the symlink in the Go binary directory, but the manifest only checks and executes notion through PATH. Include the PATH setup in this manifest guidance.

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@plugins/notion-cli/plugin.json` around lines 14 - 15, Update the installation guidance in the plugin manifest around the notion-cli install and symlink commands to explicitly add the resolved Go binary directory to PATH, ensuring the installed notion executable can be found by subsequent PATH-based checks and invocations.

javimosch · 2026-08-08T05:42:43Z

🤖 Escalated to CEO: https://github.com/javimosch/am-fleet/issues/45

The rebaser could not resolve conflicts automatically. Reply on the issue above to unblock this PR.

coderabbitai Bot reviewed Aug 4, 2026

View reviewed changes

This was referenced Aug 5, 2026

fix(#298): add go install instructions to bundled notion-cli plugin #373

Open

fix(#298): add go install instructions to bundled notion-cli plugin #374

Open

fix(#298): add go install instructions to bundled notion-cli plugin #375

Merged

Navigation Menu

fix(#298): add go install instructions to bundled notion-cli plugin - #372

fix(#298): add go install instructions to bundled notion-cli plugin#372
javimosch wants to merge 1 commit into
masterfrom
am/am-f17c27-dkg9sc8fjw7x-91e45236

javimosch commented Aug 4, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

chatgpt-codex-connector Bot commented Aug 4, 2026

Uh oh!

coderabbitai Bot commented Aug 4, 2026 •

edited

Loading

Walkthrough

Changes

Uh oh!

coderabbitai Bot left a comment

Uh oh!

coderabbitai Bot Aug 4, 2026

Uh oh!

coderabbitai Bot Aug 4, 2026

Uh oh!

javimosch commented Aug 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

		"go install github.com/4ier/notion-cli@latest",
		"GOBIN=$(go env GOBIN); GOPATH=$(go env GOPATH); ln -sf \"${GOBIN:-$GOPATH/bin}/notion-cli\" \"${GOBIN:-$GOPATH/bin}/notion\"",

Conversation

javimosch commented Aug 4, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by CodeRabbit

Uh oh!

chatgpt-codex-connector Bot commented Aug 4, 2026

Uh oh!

coderabbitai Bot commented Aug 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Aug 4, 2026

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot Aug 4, 2026

Choose a reason for hiding this comment

Uh oh!

javimosch commented Aug 8, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

javimosch commented Aug 4, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Aug 4, 2026 •

edited

Loading