Repository navigation
Conversation
0thernet
changed the title
Part of the hraness velocity + cost sweep (lane L-tail-4). types is
public, so its minutes are free. This PR is about wall-clock time,
merge-flow correctness and keeping the baseline consistent.
## What changed
- **CI concurrency**: the group is `ci-${{ github.event_name ==
'pull_request' && github.ref || github.sha }}` with `cancel-in-progress:
${{ github.event_name == 'pull_request' }}`. A new push to a PR still
cancels that PR's superseded run. Before this change, a later push to
`main` could cancel an earlier main run (`cancel-in-progress: true` on
`refs/heads/main`). Main runs are now grouped per commit and never
cancelled.
- **Bun cache** (audit F9): `actions/cache/restore` on
`~/.bun/install/cache`, keyed on `bun.lock`. `actions/cache/save` runs
only on pushes to `main`, so PR runs cannot churn or evict the cache.
- **Dependabot** (audit F5): new `.github/dependabot.yml` for
github-actions. It runs weekly, puts all updates in one group, and
allows at most 2 open PRs. This closes the hraness-ci baseline gap: the
file was 404 before.
The `Required` check and its workflow triggers are unchanged: no
`paths:` filter was added. `release.yml` is unchanged.
## Evidence (before)
CI wall time, recent runs:
- 36484235142 (push): 31 s
- 36484113849 (PR): 40 s
- 36395445695 (push): 37 s
- 36391156902 (PR): 33 s
- 36372381598 (push): 37 s
- 36372180137 (PR): 29 s
Billed minutes: 0 (public repo). Install takes under 3 s, so the cache
saving is small (a few seconds per run).
## Expected after
- Wall time about the same, possibly a few seconds less once main has
saved a cache.
- No cancelled main verifications.
- GitHub Actions pins kept current through one grouped weekly Dependabot
PR.
Not in this PR: parked PR #9 (Ben decides whether to close it) and a
Dependabot auto-merge workflow.
Local check: `actionlint` passes.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
---------
Co-authored-by: 0thernet <894119+0thernet@users.noreply.github.com>
Co-authored-by: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
|
Reviewed during the October 1 Hraness stale-PR sweep: keep this proposal for follow-up. The newer #12 rollout provides v0.24.0 local search, but still invokes Bunx and installs no local Wordcell dependency. This PR uniquely removes package installation from the first query after frozen setup. Its v0.22 dependency and hosted-default script are stale and must be adapted to current main. Next step: Keep for a focused rebase to the current release and local selected-passage default, preserving the no-runner acceptance evidence and refreshing the lockfile. Assessment used head |
|
Completed against current main in #19, merged as |
1 participant
Adds repository KB search using pinned Wordcell 0.22.0, with hosted reranking for public queries and an explicit local-only command. Wordcell is a development dependency installed by the frozen setup step; search scripts invoke its local executable, eliminating the cold Bunx installation from the query path. Existing KB maintenance commands and production dependencies are preserved.
Validation: frozen Bun install and
bun run checkpassed. Packed-consumer typechecks passed under Bundler and NodeNext; all 4 tests passed. The first local search with an empty package cache completed in 1933 ms with OS-enforced network denial, literal query forwarding, no package-manager invocation, and unchanged KB files. Current-head CI remains required before merge.