Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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 .agents/skills/agentation-release/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,8 @@ Read `references/troubleshooting.md` if the release workflow fails.
Run these commands from the repo root:

```bash
pnpm --filter @alexgorbatchev/agentation build
pnpm --filter feedback-tool-example build
bun run build:agentation
bun run --filter ./packages/example build
```

Run additional targeted tests when the change touches runtime behavior.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,16 +9,16 @@

## Known failure modes

### `pnpm/action-setup` version conflict
### `oven-sh/setup-bun` version drift

Symptom:

- The workflow complains that pnpm is specified both in the action and in `package.json#packageManager`.
- CI or release behaves differently locally vs GitHub Actions because Bun versions do not match.

Fix:

- Let `pnpm/action-setup` read the version from `package.json#packageManager`.
- Do not add a separate `version:` field in `.github/workflows/release.yml` unless you also remove the package-manager pin.
- Keep `.github/workflows/release.yml` and `package.json#packageManager` pinned to the same Bun version.
- Regenerate `bun.lock` with that Bun version when dependency metadata changes.

### npm provenance `E422` repository mismatch

Expand Down
4 changes: 2 additions & 2 deletions .agents/skills/agentation-vercel/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ Run the docs build from a clean package-output state before touching Vercel:

```bash
rm -rf packages/agentation/dist
pnpm --filter feedback-tool-example build
bun run --filter ./packages/example build
```

If the local clean build fails, fix that first.
Expand All @@ -59,7 +59,7 @@ Prefer `vercel inspect <deployment-url> --logs` for build failures.

- `Module not found: Can't resolve '@alexgorbatchev/agentation'`
- The workspace package was not built before `next build`.
- Preserve the `prebuild` hook in `packages/example/package.json`.
- Preserve the `prebuild` hook in `packages/example/package.json`, keep it Bun-based, and make sure it hops to the repo root before filtering workspaces.

- `The vercel.json file should be inside of the provided root directory.`
- The config file is in the wrong directory for this project.
Expand Down
6 changes: 3 additions & 3 deletions .agents/skills/agentation-vercel/references/setup.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@

`packages/example/package.json` must keep this sequence:

1. `prebuild`: `pnpm --filter @alexgorbatchev/agentation build`
1. `prebuild`: `cd ../.. && bun run --filter ./packages/agentation build`
2. `build`: `next build`

That prebuild is required because the docs app imports the workspace package and the package exports from `packages/agentation/dist/*`.
Expand All @@ -23,7 +23,7 @@ Use this exact clean-build check when diagnosing Vercel:

```bash
rm -rf packages/agentation/dist
pnpm --filter feedback-tool-example build
bun run --filter ./packages/example build
```

If this fails, Vercel will fail too.
Expand All @@ -32,7 +32,7 @@ If this fails, Vercel will fail too.

- Do not add a repo-root `vercel.json` while the Vercel project root remains `packages/example`.
- Do not reintroduce `packages/agentation/vercel.json`; it is stale for this monorepo.
- Keep `package.json#packageManager` aligned with the pnpm version expected by CI and Vercel.
- Keep `package.json#packageManager` aligned with the Bun version expected by CI and Vercel.

## Expected warnings

Expand Down
18 changes: 14 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,23 @@ jobs:
with:
fetch-depth: 0

- uses: pnpm/action-setup@v4
- uses: oven-sh/setup-bun@v2
with:
bun-version: 1.3.10

- uses: actions/setup-node@v6
with:
node-version: 24
cache: pnpm
registry-url: https://registry.npmjs.org

- name: Cache Bun dependencies
uses: actions/cache@v5
with:
path: ~/.bun/install/cache
key: ${{ runner.os }}-bun-${{ hashFiles('**/bun.lock') }}
restore-keys: |
${{ runner.os }}-bun-

- name: Derive release version from tag
run: |
set -euo pipefail
Expand All @@ -45,10 +55,10 @@ jobs:
EOF

- name: Install dependencies
run: pnpm install --frozen-lockfile
run: bun install --frozen-lockfile

- name: Build package
run: pnpm --filter @alexgorbatchev/agentation build
run: bun run build:agentation

- name: Publish package
working-directory: packages/agentation
Expand Down
26 changes: 15 additions & 11 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,22 +10,26 @@ jobs:
ci:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/checkout@v6

- uses: pnpm/action-setup@v4
- uses: oven-sh/setup-bun@v2
with:
version: 10
bun-version: 1.3.10

- uses: actions/setup-node@v4
- name: Cache Bun dependencies
uses: actions/cache@v5
with:
node-version: 22
cache: pnpm
path: ~/.bun/install/cache
key: ${{ runner.os }}-bun-${{ hashFiles('**/bun.lock') }}
restore-keys: |
${{ runner.os }}-bun-

- run: pnpm install --frozen-lockfile
- run: bun install --frozen-lockfile

- name: Install Playwright browsers
run: pnpm --filter @alexgorbatchev/agentation exec playwright install --with-deps chromium
working-directory: packages/agentation
run: bunx playwright install --with-deps chromium

- run: pnpm build
- run: pnpm build:storybook
- run: pnpm test
- run: bun run build
- run: bun run build:storybook
- run: bun run test
3 changes: 0 additions & 3 deletions .npmrc

This file was deleted.

8 changes: 4 additions & 4 deletions AGENTS.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,10 @@ A floating toolbar for annotating web pages and collecting structured feedback f
## Development

```bash
pnpm install # Install all workspace dependencies
pnpm dev # Run both package watch + website dev server
pnpm build # Build all packages
pnpm test # Run all tests
bun install # Install all workspace dependencies
bun run dev # Run both package watch + website dev server
bun run build # Build all packages
bun run test # Run all tests
```

## Important
Expand Down
14 changes: 8 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -128,24 +128,26 @@ Compared with the original upstream project, this repository includes:
## Development

```bash
pnpm install
pnpm build
pnpm test
bun install
bun run build
bun run test
```

Coverage workflow:

```bash
# Stable package coverage run (unit project)
pnpm --filter @alexgorbatchev/agentation exec vitest run --coverage
cd packages/agentation
bunx vitest run --coverage

# Explicit unit-only coverage
pnpm --filter @alexgorbatchev/agentation exec vitest run --project unit --coverage
cd packages/agentation
bunx vitest run --project unit --coverage
```

Notes:

- `pnpm test` remains the full suite entry point (including browser/storybook tests).
- `bun run test` remains the full suite entry point (including browser/storybook tests).
- Coverage runs are aligned to the supported Vitest coverage path above.

## License
Expand Down
Loading