Skip to content

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

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
javimosch wants to merge 1 commit into
base: master
Choose a base branch
from
+18 −6
Open

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

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions plugins/notion-cli/install-guidance.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@
"binary": "notion",
"check": "which notion",
"install_steps": [
"brew install 4ier/tap/notion-cli",
"go install github.com/4ier/notion-cli@latest",
"Verify: notion --version",
Comment on lines +6 to 7

@coderabbitai coderabbitai Bot Aug 3, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
set -euo pipefail

echo "Files:"
git ls-files | rg '(^|/)plugins/notion-cli/|notion-cli' || true

echo
echo "Relevant file sections:"
for f in plugins/notion-cli/install-guidance.json plugins/notion-cli/plugin.json plugins/notion-cli/skills/quickstart/SKILL.md; do
  if [ -f "$f" ]; then
    echo "--- $f ---"
    wc -l "$f"
    sed -n '1,90p' "$f"
    if [ "$f" = plugins/notion-cli/plugin.json ]; then
      echo "--- plugin.json lines 220-242 ---"
      sed -n '220,242p' "$f"
    fi
  else
    echo "missing $f"
  fi
done

echo
echo "Search notion-cli install/help references:"
rg -n "go install github\.com/4ier/notion-cli|notion --version|notion auth login|missingDependencyHelp|install_steps" plugins/notion-cli .github plugins 2>/dev/null | head -200

Repository: javimosch/supercli

Length of output: 32633

🌐 Web query:

Go installation PATH GOPATH/bin default GOBIN documentation

💡 Result:

When you run the go install command, Go compiles and installs executables into a specific directory controlled by the GOBIN and GOPATH environment variables [1][2]. The installation location is determined as follows: 1. If the GOBIN environment variable is set, binaries are installed into the directory specified by GOBIN [3][4]. GOBIN must be an absolute path [4]. 2. If GOBIN is not set, binaries are installed into the bin subdirectory of the first directory listed in the GOPATH environment variable [3][5]. 3. If neither GOBIN nor GOPATH is set, Go defaults to using $HOME/go/bin on Unix systems or %USERPROFILE%\go\bin on Windows [1][6][4]. To make these installed binaries easily executable from your command line, it is standard practice to add the binary installation directory (e.g., $HOME/go/bin) to your system PATH environment variable [3][7]. You can check your current configuration by running the command go env [3]. If you wish to change the default binary installation location, you can update GOBIN using: go env -w GOBIN=/your/custom/path/bin [3][7] Or, you can adjust the GOPATH workspace location to change where the default bin directory is located [6]. Note that while modern Go development primarily uses Go modules for dependency management, the GOPATH variable remains relevant for determining these default installation and cache locations [2][8][4].

Citations:

Make Go installation output reachable by SuperCLI.

go install writes the binary to GOBIN, or $HOME/go/bin when GOBIN is unset. Add the installation directory to PATH before verification and install_steps, and document it in version/passthrough help and quickstart installation. If users omit this step, verify: which notion, notion --version, and notion passthrough calls can fail after installation succeeds.

📍 Affects 3 files
  • plugins/notion-cli/install-guidance.json#L6-L7 (this comment)
  • plugins/notion-cli/plugin.json#L14-L14
  • plugins/notion-cli/plugin.json#L33-L33
  • plugins/notion-cli/plugin.json#L233-L233
  • plugins/notion-cli/skills/quickstart/SKILL.md#L61-L63
🤖 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 - 7, Update the Go
installation flow in plugins/notion-cli/install-guidance.json:6-7 to add the Go
binary directory to PATH before verification and install_steps, covering GOBIN
and the default $HOME/go/bin location. Update the related version/passthrough
help at plugins/notion-cli/plugin.json:14-14, plugin.json:33-33, and
plugin.json:233-233, plus the quickstart installation instructions at
plugins/notion-cli/skills/quickstart/SKILL.md:61-63, to document this PATH setup
so which notion, notion --version, and passthrough calls can locate the
installed binary.

"Authenticate: echo 'ntn_xxxxx' | notion auth login --with-token",
"Alternative auth: export NOTION_TOKEN=ntn_xxxxx",
"supercli plugins install ./plugins/notion-cli --on-conflict replace --json"
],
"note": "Also available as Go binary from GitHub Releases: https://github.com/4ier/notion-cli/releases. npm: npm install -g @4ier/notion-cli. Token stored in ~/.config/notion-cli/config.json (mode 0600). Auto-detects JSON output when piped to another command."
"note": "Primary install is go install. Also available via Homebrew (brew install 4ier/tap/notion-cli), as a Go binary from GitHub Releases (https://github.com/4ier/notion-cli/releases), or via npm (npm install -g @4ier/notion-cli). Token stored in ~/.config/notion-cli/config.json (mode 0600). Auto-detects JSON output when piped to another command."
}
6 changes: 3 additions & 3 deletions plugins/notion-cli/plugin.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
"binary": "notion",
"check": "which notion",
"install_steps": [
"brew install 4ier/tap/notion-cli",
"go install github.com/4ier/notion-cli@latest",
"Verify: notion --version",
"Authenticate: echo 'ntn_xxxxx' | notion auth login --with-token",
"supercli plugins install ./plugins/notion-cli --on-conflict replace --json"
Expand All @@ -30,7 +30,7 @@
"adapterConfig": {
"command": "notion",
"baseArgs": ["--version"],
"missingDependencyHelp": "Install notion-cli: brew install 4ier/tap/notion-cli"
"missingDependencyHelp": "Install notion-cli: go install github.com/4ier/notion-cli@latest (or brew install 4ier/tap/notion-cli)"
},
"args": []
},
Expand Down Expand Up @@ -230,7 +230,7 @@
"adapterConfig": {
"command": "notion",
"passthrough": true,
"missingDependencyHelp": "Install notion-cli: brew install 4ier/tap/notion-cli. Authenticate: echo 'ntn_xxxx' | notion auth login --with-token"
"missingDependencyHelp": "Install notion-cli: go install github.com/4ier/notion-cli@latest (or brew install 4ier/tap/notion-cli). Authenticate: echo 'ntn_xxxx' | notion auth login --with-token"
},
"args": []
}
Expand Down
14 changes: 13 additions & 1 deletion plugins/notion-cli/skills/quickstart/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -58,11 +58,23 @@ export NOTION_TOKEN=ntn_xxxxx

## Installation

```bash
go install github.com/4ier/notion-cli@latest
```

Or install with Homebrew:

```bash
brew install 4ier/tap/notion-cli
```

Or download binary from [GitHub Releases](https://github.com/4ier/notion-cli/releases).
Or install with npm:

```bash
npm install -g @4ier/notion-cli
```

Or download a binary from [GitHub Releases](https://github.com/4ier/notion-cli/releases).

## Key Features
- **Agent-friendly**: JSON output auto-detected when piped, schema-aware, URL or ID support
Expand Down
Loading