Skip to content

Commit 9d47a6a

Browse files
authored
Merge branch 'main' into claude/fix-mcpc-timeout-YImuH
2 parents 27e1ab9 + 9f927b4 commit 9d47a6a

File tree

2 files changed

+33
-6
lines changed

2 files changed

+33
-6
lines changed

.github/workflows/ci.yml

Lines changed: 28 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -35,14 +35,14 @@ jobs:
3535
- name: Unit tests
3636
run: npm run test:unit
3737

38-
e2e:
39-
name: E2E tests
38+
e2e-node:
39+
name: E2E tests (Node.js)
4040
runs-on: ubuntu-latest
4141

4242
steps:
4343
- uses: actions/checkout@v6
4444

45-
- name: Set up Node.js 22
45+
- name: Set up Node.js 24
4646
uses: actions/setup-node@v6
4747
with:
4848
node-version: 24
@@ -54,13 +54,35 @@ jobs:
5454
- name: Build
5555
run: npm run build
5656

57-
- name: Install Bun
58-
uses: oven-sh/setup-bun@v2
59-
6057
- name: E2E tests
6158
# Run without gnome-keyring: @napi-rs/keyring fails fast when no D-Bus secret
6259
# service is available, and keychain.ts automatically falls back to encrypted
6360
# file storage (~/.mcpc/credentials.json, mode 0600). The keychain code paths
6461
# are covered by unit tests; e2e tests only need credentials to work, not a
6562
# specific storage backend.
6663
run: ./test/e2e/run.sh --no-build --parallel 4
64+
65+
e2e-bun:
66+
name: E2E tests (Bun)
67+
runs-on: ubuntu-latest
68+
69+
steps:
70+
- uses: actions/checkout@v6
71+
72+
- name: Set up Node.js 24
73+
uses: actions/setup-node@v6
74+
with:
75+
node-version: 24
76+
cache: npm
77+
78+
- name: Install Bun
79+
uses: oven-sh/setup-bun@v2
80+
81+
- name: Install dependencies
82+
run: npm ci
83+
84+
- name: Build
85+
run: npm run build
86+
87+
- name: E2E tests
88+
run: ./test/e2e/run.sh --no-build --runtime bun --parallel 4

CLAUDE.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -517,6 +517,11 @@ Example: `mcpc @apify logging-set-level debug`
517517

518518
## Common Implementation Patterns
519519

520+
After making any code changes, always run `npm run lint` and fix all errors before finishing. The lint command checks both ESLint rules and
521+
Prettier formatting; run `npx prettier --write` on any files flagged for formatting issues or `npm run lint` to fix the entire repo.
522+
523+
For any non-trivial change (new feature, bug fix, behaviour change, or notable refactor), add an entry to the `[Unreleased]` section of `CHANGELOG.md` before finishing. Use the appropriate category (`Added`, `Changed`, `Fixed`, `Removed`). Skip purely internal changes such as test-only edits or code style fixes.
524+
520525
When implementing features:
521526

522527
1. **Keep core runtime-agnostic** - Use native APIs, avoid runtime-specific dependencies

0 commit comments

Comments
 (0)