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
25 changes: 3 additions & 22 deletions .github/workflows/pr-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@ name: pr-build
on:
pull_request:
branches: [main]
merge_group:
workflow_dispatch:
push:
branches: [main]

Expand Down Expand Up @@ -32,9 +34,6 @@ jobs:
node --check packages/linear-agent-hooks/lib/transcript.mjs
node --check packages/linear-agent-hooks/bin/setup.mjs
node --check packages/issue-standards/bin/setup.mjs
node --check scripts/project-acceptance.mjs
node --check scripts/check-llm-cost-catalog-api-alignment.mjs
node --check test/project-acceptance/llm-cost-architecture-contracts/catalog-api-alignment.check.mjs
node --check packages/llm-cost-attribution/bin/llm-cost.mjs
for f in packages/llm-cost-attribution/src/*.mjs packages/llm-cost-attribution/src/transcripts/*.mjs; do
node --check "$f"
Expand All @@ -48,22 +47,4 @@ jobs:
run: npm install --no-audit --no-fund

- name: Run tests
run: |
# Project ATDD checks live under test/project-acceptance/ and run only
# in project-acceptance.yml while that temporary gate is active.
node --test \
packages/linear-agent-hooks/tests/post-tool-use.test.mjs \
packages/linear-agent-hooks/tests/stop.test.mjs \
packages/linear-agent-hooks/tests/comment.test.mjs \
packages/linear-agent-hooks/tests/transcript.test.mjs \
scripts/check-llm-cost-catalog-api-alignment.test.mjs \
packages/llm-cost-attribution/test/issue-pattern.test.mjs \
packages/llm-cost-attribution/test/aggregator.test.mjs \
packages/llm-cost-attribution/test/usage-jsonl.test.mjs \
packages/llm-cost-attribution/test/transcript-to-usage.test.mjs \
packages/llm-cost-attribution/test/pricing.test.mjs \
packages/llm-cost-attribution/test/multi-issue.test.mjs \
packages/llm-cost-estimation/test/enrich.test.mjs \
packages/llm-cost-estimation/test/smoke.test.mjs \
packages/llm-cost-estimation/test/cli.test.mjs \
packages/llm-cost-estimation/test/no-org-data.test.mjs
run: npm test
27 changes: 0 additions & 27 deletions .github/workflows/project-acceptance.yml

This file was deleted.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
"name": "groove",
"private": true,
"scripts": {
"project-acceptance": "node scripts/project-acceptance.mjs"
"test": "node --test test/*.test.mjs && npm test --workspaces --if-present"
},
"workspaces": [
"packages/*"
Expand Down
3 changes: 3 additions & 0 deletions packages/linear-agent-hooks/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,9 @@
"bin": {
"linear-agent-hooks": "./bin/setup.mjs"
},
"scripts": {
"test": "node --test tests/*.test.mjs"
},
"files": [
"bin/",
"hooks/",
Expand Down
2 changes: 1 addition & 1 deletion packages/llm-cost-attribution/docs/use-cases.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ port-based workflows live in `src/attribution-workflow.mjs` (core, I/O-free); th
real Claude/Codex transcript and usage-JSONL adapters live in
`src/attribution-adapters.mjs`. Boundary rule: core modules import no filesystem,
transcript, usage-JSONL, CLI, HTTP/Linear, or child_process APIs — enforced by
`npm run test:boundary` and the project-acceptance boundary check.
`npm run test:boundary`.

## CreateAttributionWorkflow
`createAttributionWorkflow({ sessionSource, issueMatcher, usageRecordSource, usageRecordSink, recordedAt })`
Expand Down
Loading
Loading