Skip to content

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

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

Closed
javimosch wants to merge 1 commit into from
Closed

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

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.

🩺 Stability & Availability | 🟠 Major | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

command -v go >/dev/null
gobin="$(go env GOBIN)"
if [ -z "$gobin" ]; then
  gobin="$(go env GOPATH)/bin"
fi

case ":${PATH}:" in
  *":${gobin}:"*) ;;
  *) printf 'Go binary directory is not on PATH: %s\n' "$gobin" >&2; exit 1 ;;
esac

Repository: javimosch/supercli

Length of output: 214

🏁 Script executed:

#!/usr/bin/env bash
set -euo pipefail

printf 'GOMODCACHE: '
go env GOMODCACHE
printf 'GOBIN: %s\n' "$(go env GOBIN || true)"
printf 'GOPATH: %s\n' "$(go env GOPATH || true)"
printf 'PATH includes standard Go bin? '
gobin="$(go env GOBIN || true)"
if [ -z "$gobin" ]; then
  gobin="$(go env GOPATH)/bin"
fi
case ":${PATH}:" in
  *":${gobin}:"*) echo yes ;;
  *) echo no; printf 'actual PATH contains: ' ; sed -n 's#:/##p' <<< "$PATH" | tr ':' '\n' | head -20 ;;
esac

printf '\nGo toolchain availability:\n'
command -v go || true
go version 2>/dev/null || true

printf '\nRelevant file excerpts:\n'
for f in plugins/notion-cli/install-guidance.json plugins/notion-cli/plugin.json plugins/notion-cli/skills/quickstart/SKILL.md; do
  echo "--- $f"
  sed -n '1,260p' "$f"
done

Repository: javimosch/supercli

Length of output: 12426

Document the go install binary directory in the installation guidance.

go install places binaries in $(go env GOBIN || $(go env GOPATH)/bin)/bin, and that directory is not documented on the installation surfaces. Since the installed binary must satisfy which notion, add the exported Go bin path as the install step before notion --version, or use an alternative install method, and include the same step in install-guidance.json, plugin.json fallback install, and skills/quickstart/SKILL.md.

📍 Affects 3 files
  • plugins/notion-cli/install-guidance.json#L6-L7 (this comment)
  • plugins/notion-cli/plugin.json#L14-L15
  • 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, Document and
export the Go binary directory before verifying the installed command: update
plugins/notion-cli/install-guidance.json lines 6-7,
plugins/notion-cli/plugin.json lines 14-15, 33, and 233, and
plugins/notion-cli/skills/quickstart/SKILL.md lines 61-63. Ensure each
installation surface adds the Go bin path to PATH before the notion --version or
which notion check, while preserving the existing fallback installation
guidance.

"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
8 changes: 7 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,17 @@ 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 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