Skip to content

fix: build dashboard SPA in CI/publish and lead install with npm proof path #46

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
WellDunDun wants to merge 1 commit into from
+17 −4
Closed

fix: build dashboard SPA in CI/publish and lead install with npm proof path #46

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
10 changes: 10 additions & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,16 @@ jobs:
- run: bunx @biomejs/biome check .
- run: bun run lint-architecture.ts

build-dashboard:
runs-on: ubuntu-latest
permissions:
contents: read
steps:
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
- uses: oven-sh/setup-bun@ecf28ddc73e819eb6fa29df6b34ef8921c743461 # v2
- run: bun install
- run: bun run build:dashboard

test:
runs-on: ubuntu-latest
permissions:
Expand Down
3 changes: 3 additions & 0 deletions .github/workflows/publish.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,9 @@ jobs:
- name: Install dependencies
run: bun install

- name: Build dashboard SPA
run: bun run build:dashboard

- name: Verify npm version for trusted publishing
run: npm --version

Expand Down
8 changes: 4 additions & 4 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,17 @@ Built for **Claude Code**. Also works with Codex, OpenCode, and OpenClaw. Zero r
## Install

```bash
npx skills add selftune-dev/selftune
npx selftune@latest doctor
```

Then tell your agent: **"initialize selftune"**
That's it. One command to install and verify. Then tell your agent: **"initialize selftune"**
Comment on lines +33 to +36
Copy link

@coderabbitai coderabbitai bot Mar 14, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Wording may mislead users about installation.

Line 36 says "One command to install and verify" but npx selftune@latest doctor runs the package temporarily via npx without persisting it. The actual installation for Claude Code users is the skill-based approach on line 43. Consider rewording to something like "One command to verify your setup" or clarifying that this tests the npm package without permanent installation.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 33 - 36, The phrase "One command to install and
verify" is misleading because `npx selftune@latest doctor` runs the package
temporarily; update the README sentence that follows the `npx selftune@latest
doctor` example to indicate this is a verification/test run rather than a
permanent install (e.g., change to "One command to verify your setup" or add
"runs via npx without persisting install"), and optionally add a short pointer
mentioning the actual installation method referenced later (the skill-based
approach) so readers know where to go for a persistent install.


Two minutes. No API keys. No external services. No configuration ceremony. Uses your existing agent subscription. Within minutes you'll see which skills are undertriggering.

**CLI only** (no skill, just the CLI):
**As a skill** (recommended for Claude Code users):

```bash
npx selftune@latest doctor
npx skills add selftune-dev/selftune
```

## Before / After
Expand Down
Loading