Files
PaperClipAI/doc/ci-profiling.md
T
DottaandPaperclip 307eb16c0c perf(ci): avoid duplicate extraction in public install checks (#16460)
## Thinking Path

> - Paperclip manages AI agents for work.
> - Contributors wait for CI before they can deliver changes.
> - After #16434, Build is the longest CI job.
> - Its clean public npm install spends about 90 seconds installing
packed artifacts.
> - npm extracts local archives to read metadata before it installs
them.
> - Serve metadata from those same packed artifacts, then let normal npm
verify and install their bytes once.
> - Preserve the offline lifecycle and runtime checks, and measure the
complete workflow.

## Linked Issues or Issue Description

Refs #16434 and #16429.

**Current behavior:** The Linux Build check installs every staged
package by local tarball path. npm's file fetcher extracts an archive to
obtain its manifest. Installation then extracts it again. In the last
green run, Build took 408 seconds; consumer installation took 89.7
seconds. The full workflow took 8m06s.

**Proposed behavior:** Read manifests directly from the packed archives.
Serve those manifests and their SHA-512-bound tarballs from a temporary
registry on container loopback. Use normal npm installation into the
empty consumer. Other packages still resolve from the public npm
registry.

**Alternatives:** More parallel jobs would add setup and transfer work.
Skipping the install or runtime probes would remove useful coverage.
This change removes duplicate extraction instead. It does not add a
product feature or duplicate a roadmap item.

## What Changed

- Build a registry catalog from the actual tarball manifests and archive
hashes.
- Run the short-lived registry and npm in the existing unprivileged
download container. No host port is exposed.
- Preserve public dependency tarball URLs, archive integrity checks,
disabled initial lifecycle scripts, the offline rebuild, lock checks and
all Codex, Grok and Pi probes.
- Close the registry before offline verification. Keep native ARM and
Mac install qualification unchanged.
- Add real npm parity and failure tests. Record metadata preparation
time in the existing timing receipt.
- Fix an existing policy false positive: the filename
`docker-compose-secrets.test.mjs` is not a workflow credential. Inspect
GitHub expressions and retain rejection of Honeycomb keys, dot/bracket
secret reads and `toJSON(secrets)`.

## Verification

- 31 focused install-contract checks pass on Node 24.21.0. All 16
targeted CI profile-policy tests also pass.
- Real npm file and registry installs produce identical installed files,
bundled dependencies, optional dependencies and executable links. Both
defer lifecycle execution; offline rebuild works after the registry
closes and preserves the lock.
- npm rejects an archive whose gzip header changes while its extracted
payload remains valid. The CLI propagates the failure.
- The permanent real-npm parity test now includes non-catalog
`is-number@7.0.0`. Both paths install it from its official npmjs
tarball, and the test asserts the lock URL. Fetches have a 10-second
bound and no retries. This addresses the first Greptile finding. A
separate live probe also confirmed public provenance.
- The first local run hit the host sandbox's loopback `EPERM`; the same
checks passed with local listener access. An initial corrupted-gzip
negative fixture hit decompression failure before integrity
verification; the final fixture preserves valid gzip and specifically
proves integrity rejection. An old command-shape assertion was updated
for the new entry point without removing its ordering checks.
- The [first CI
run](https://github.com/paperclipai/paperclip/actions/runs/38778371043)
passed all test lanes and Build, but the existing credential-guard false
positive failed policy and both aggregates. Its 7m25s full duration is
**not** counted as a passing performance sample. The guard repair and
public-download regression are on the new head.
- The real Linux install passed all 18 packed packages, the lifecycle
sentinel, offline rebuild, unchanged lock, official Codex payload/lease
checks, Grok missing/present checks and Pi admission. The receipt
confirms `artifactTransport: loopback-registry`. No provider calls ran.
- Compared with the previous green run, consumer install fell **89.72s →
66.29s** (26.1%), including 1.87s to prepare metadata. Build fell **408s
→ 374s**. Both Build hosts were `m7i-flex.xlarge`; these are separate
runs, so registry/cache and upstream changes remain confounders.
- The base advanced since the previous 8m06s run, including runtime and
test changes. Treat that duration as historical context, not an
identical-source control.
- Pre-conflict head `d8f9de1c` [passes all 48 workflow
jobs](https://github.com/paperclipai/paperclip/actions/runs/38779549190),
including both required branch-protection checks. Greptile is **5/5** on
this head; its review thread is resolved. The non-required Superagent
Security Scan is still running.

| Measurement | Previous green run | Final green run | Observed change |
| --- | ---: | ---: | ---: |
| Full CI, workflow creation through final aggregate | 8m06s | **7m35s**
| **31s shorter (6.4%)** |
| Build job execution | 408s | **389s** | **19s shorter** |
| Clean consumer install, including catalog preparation | 89.72s |
**69.69s** | **20.03s less (22.3%)** |

The final catalog preparation took 1.87s and is included in the install
figure. Both Build hosts were `m7i-flex.xlarge`. Build began 42 seconds
after workflow creation instead of 53 seconds in the historical run, so
faster queue/setup admission explains part of the 31-second total.
Upstream code also advanced; these measurements are not a controlled
causal estimate of the complete gain. The two candidate install
measurements (66.29s and 69.69s) support the narrower reduction in
install work. No tests or runtime probes were removed.

- Conflict-resolution head `ea8287b81` [passes all 48 workflow
jobs](https://github.com/paperclipai/paperclip/actions/runs/38789785045),
including both required gates. Greptile re-reviewed this exact head at
5/5, with the sole earlier thread resolved. The conflict was the
independent filename-policy fix; the stronger expression guard is
retained. This run took **8m04s** overall, Build 423s, consumer
installation 80.56s and bundled-server staging 31.17s. This confirms
meaningful run variance: the earlier 7m35s is a sample, not a guaranteed
new wall-clock duration.

- A second independent master update touched the same credential test.
Final head `1c740b73e` resolves that conflict while retaining the same
stronger guard. [All 48 jobs
pass](https://github.com/paperclipai/paperclip/actions/runs/38792983313);
Greptile re-reviewed this exact head at 5/5. This run took **7m32s**
overall, Build **385s**, consumer install **67.62s**. Auto-merge is
enabled but GitHub requires code-owner approval.

## Risks

The initial install now uses registry package specs instead of file
specs. The parity test checks the resulting installed bytes. Full CI
must still exercise real bundled provider graphs and all offline probes.
The temporary consumer lock contains loopback artifact URLs; it is not
exported or reused for another installation. Native release consumers
retain their existing artifact install path. Queue, host and cache
variance limit attribution from separate workflow runs.

## Model Used

OpenAI GPT-6 through Codex, with repository inspection, execution and
GitHub tools. The serving model suffix and context capacity are not
exposed to this session.

## Checklist

- [x] I have included a thinking path that traces from project context
to this change
- [x] I have specified the model used (with version and capability
details)
- [x] I have checked ROADMAP.md and confirmed this PR does not duplicate
planned core work
- [x] I have searched GitHub for duplicate or related PRs and linked
them above
- [x] I have either linked existing issues or described the issue in
this PR
- [x] I have not referenced internal or instance-local Paperclip issues
or links
- [x] My branch name describes the change and contains no private ticket
identifier
- [x] I have run tests locally and they pass
- [x] I have added or updated tests where applicable
- [x] I have updated relevant documentation
- [x] I have considered and documented the risks
- [x] All Paperclip CI gates are green (48 jobs and both required gates
on conflict-resolution head)
- [x] Greptile is 5/5 with no open P2s, recommendations or follow-ups
- [x] I will address all Greptile and reviewer comments before
requesting merge

---------

Co-authored-by: Paperclip <noreply@paperclip.ing>
2026-10-15 14:52:15 -05:00

13 KiB

Profile the complete PR suite

Add the profile-ci label to a PR authored by a CI CODEOWNER. Adding the label starts a fresh PR workflow run. Further pushes while the label remains also run with profiling enabled. The run is named CI profile · PR #number. It runs all lanes of the main PR suite, including middle-of-stack and release-notes-only PRs that normally skip most lanes.

After the run finishes, open its Summary and follow Open the Honeycomb waterfall. The separate CI profile workflow exports the trace after all jobs finish. Its summary has the exact time range, job/span counts, and coverage warnings. Keep the label on the PR and its head unchanged until export finishes. Remove the label to stop profiling subsequent pushes. To profile the same revision again, remove and re-add the label or rerun all jobs.

The PR author, original workflow actor, and rerun actor must all be individual users in the trusted master branch's .github/** CODEOWNERS rule. GitHub user IDs are resolved from that rule; PR edits to CODEOWNERS cannot grant access. A label on an outside contributor's PR does not enable profiling, even when a maintainer applies it. The PR still runs normal CI and needs no Honeycomb access. Failed identity lookups disable profiling.

Ordinary PRs do not write timing artifacts or export traces. The exporter job is skipped for ordinary run names. Adding another label does not run the CI matrix.

Profiling preserves the existing trusted AWS runner routing. Eligible PR jobs use paperclip-public-pr-x64; the routing gate uses paperclip-ci-routing-x64. The existing fail-closed GitHub-hosted fallback still applies if AWS routing cannot be authorized. Only the separate post-run Honeycomb exporter uses ubuntu-latest directly; it is outside the measured PR suite.

Read the waterfall

The workflow root measures elapsed time from run creation to the last job's completion. For reruns, it starts at that attempt's run_started_at instead of the original creation time. Every job is a child on the same wall clock, with its steps and available test details beneath it. All spans use a sample rate of one. There is no sampling within the selected attempt. For a compact Gantt view, open a job row's menu and choose Collapse spans at this depth. Expand an individual job to inspect its steps and test files.

The expandable hierarchy is workflow → job → step → test filename → tests. Vitest, Playwright, and Node file rows use repository-relative filenames, also available in test.file. Separate files retain their own children even when individual test names repeat across files. Honeycomb shows only the 32,000 spans closest to the root in a very large trace. All file rows are near the root, but the chart can omit some individual test rows. Use View events to query all recorded test spans. Test names are limited to 256 characters in the trace; file paths remain intact.

  • Compare the jobs that finish last (ci.finishes_last) with other overlapping work. A long job that finishes early is not necessarily delaying CI.
  • Expand setup, dependency installation, cache restore, Rust compilation, builds, and tests to find repeated or slow work.
  • ci.execution_ms measures a job's execution. ci.start_offset_ms shows when it started relative to the whole run. Do not add parallel job durations to calculate elapsed time.
  • Waiting spans cover job creation to runner start. Dependency waits can occur before GitHub creates a job, so they appear as gaps from the workflow start. GitHub does not expose a separate ready timestamp; do not interpret every gap as runner queue time or claim this is a computed dependency critical path.

GitHub job/step timestamps have second precision. Vitest and Playwright record test timestamps at the runner. Node's test reporter records completion receipt time and duration (test.clock=reporter-receipt); buffered IPC delivery can shift those small test spans. Vitest file spans include its test/hook execution and carry collection/setup durations as fields. Playwright file spans cover the first through last test attempt in that file. Rust and standalone script checks retain their full job/step timings; stable libtest does not expose per-test wall timestamps. The trace is CI timing data, not application tracing.

Scope includes all main PR lanes: policy, typecheck/release registry, general and serialized tests, Runner checks, Docker context checks, build, canary dry run, browser E2E, and aggregate checks. Separate Storybook, image, deployment, release, and live-provider eval workflows are outside this trace.

Failure, coverage, and reruns

The exporter uses GitHub's attempt-specific job API and paginates jobs and artifacts. It includes failed and cancelled jobs and uploads available test timings with always(). A force-cancelled or failed setup job may leave no artifact. Coverage warnings and ci.profile_complete=false make missing artifacts, interrupted reporters, rejected data, and absent test details visible. Test launchers write an expected-invocation record before starting each Vitest, Playwright, or Node process. Each reporter must write matching start/end records, so a missing reporter cannot hide behind another successful invocation in the same job. Direct package scripts can start a reporter without the launcher; their start/end pair proves collection. Compare ci.expected_invocations with ci.reported_invocations. Failed or cancelled workflows are conservatively incomplete because a failed command can prevent later tests or steps from running.

Each attempt gets a different trace ID. Rerunning failed jobs alone produces an attempt trace for those jobs; it does not replay completed jobs. Reruns are marked ci.partial_rerun=true conservatively. Use a fresh label-triggered run for an unambiguous full-suite baseline. GitHub may carry earlier successful jobs into the new attempt's job inventory. Jobs completed before the new attempt started are excluded and counted in ci.carried_job_count.

The exporter sends bounded batches with pauses and retries only events that Honeycomb rejects temporarily. If export still fails, inspect CI profile before rerunning it. A partial HTTP export may already have sent some spans; reruns use the same IDs but the Events API does not promise deduplication. Timing artifacts remain available for 14 days. Invalid/oversized artifacts stop export instead of silently truncating the suite. No logs, environment dumps, error messages, stack traces, or credentials are included in the exported field allowlist.

Operator setup and activation

Use the existing Honeycomb paperclip team, test environment, and a separate service dataset named paperclip-ci. Create GitHub environment ci-profiling with Selected branches and tags: allow only the branch master, with no PR-ref or tag patterns. Store environment secret HONEYCOMB_CI_API_KEY with a dedicated ingest-only key for the Honeycomb test environment. Do not create a repository- or organization-level secret of that name: the environment restriction prevents in-repository PR workflows from requesting the key. Tests still execute the PR branch; only the separate exporter runs from master. It needs event ingest permission; dataset creation permission is needed only if the dataset has not been created yet. Do not store the environment configuration key in CI. Create the repository label profile-ci if it does not exist.

ci-profile.yml runs from the trusted default-branch revision. It checks out only the exporter, installs no dependencies, and parses PR artifacts as bounded JSON without extracting or executing their contents. The PR jobs never receive the Honeycomb key. The exporter independently verifies CI CODEOWNER identities, the current label, repository, and exact tested PR revision before reading artifacts or sending data. Its credential is sent only in an HTTPS header to api.honeycomb.io; redirects are rejected. Exported fields are allowlisted. CODEOWNERS also covers scripts/ci-profile/**, so changes to the privileged exporter require infrastructure-owner review.

The new trusted workflow and routing definitions must merge to master before the label-triggered journey is active. A PR uses pr-trusted.yml@master and ci-routing.yml@master; do not change those trust references just to test this feature before merge.

Local qualification

node --test .github/scripts/tests/ci-profile.test.mjs .github/scripts/tests/ci-routing.test.mjs
node --test scripts/ci-profile/reporters.test.mjs
actionlint .github/workflows/{pr,pr-trusted,ci-routing,ci-profile}.yml
node scripts/ci-profile/export.mjs --run-id=RUN_ID --attempt=1 --historical --dry-run --output=/tmp/ci-trace.json

--historical is local-only and permits a deliberate export of an existing unlabeled run. It has complete GitHub job/step timing but cannot reconstruct test details that were never recorded. Omit --dry-run and supply HONEYCOMB_CI_API_KEY to send the selected baseline to Honeycomb. It is labeled as historical/incomplete test coverage; it is not proof of the label-triggered workflow.

References: Honeycomb trace fields, Events API, large trace display limit, GitHub workflow_run.

Qualify reductions in CI work

For changes to the stable test launcher, dependency setup, or the large Git and readiness fixtures, profile-ci also runs CI work benchmark. It uses one GitHub-hosted runner without secrets or AWS access. It compares the unchanged PR base and candidate sequentially. It records complete command times, test inventories, revisions, and failure logs in the ci-work-benchmark artifact. The complete general-server shard must retain the same cases. The fixture comparison retains the database contract and splits large Git scanning from bounded staging. These command timings exclude queue time and setup; use the full Honeycomb waterfall to assess the complete CI critical path.

The benchmark also exercises cold dependency setup and a real materialized cache restore after removing the installed directories. Its two qualification entries are scoped to the PR merge ref and a separate checkout path. They cannot populate master's caches. Normal PR lanes restore only; the exact canonical master push in release verification is the normal cache writer.

Materialized dependency keys include every tracked package manifest, lockfile, workspace configuration, patches, repository scripts, Node version/ABI, platform, architecture, libc, pnpm version, store location, and physical checkout path. There are no prefix restores. An input change takes the normal install path. A stale lockfile uses the existing resolution fallback but cannot publish an entry. An additional workspace install/prepare hook disables materialized reuse until its inputs are modeled. Shared/SDK build outputs have a separate key that also includes both packages' source/configuration and the build verifier. The existing verifier checks source and output fingerprints and rebuilds incomplete or changed output. Tests and their results are never cached or omitted by this mechanism.

Serialized shards share one Vitest controller with forks, isolate, one worker, and file parallelism disabled. Each file gets a separate process and test home/configuration. The isolation regression runs files in both orders and checks process IDs, environment, globals, fake timers, and filesystem state.

The readiness regression uses a real one-second PostgreSQL idle timeout and verifies that the backend connection changes while the parent write can commit. Its configured readiness window stays long. The Git regression keeps all four filename lanes above 32 MiB, while the staging case transfers and deletes 4,000 files whose filename list exceeds 2 MiB. It still checks every staged file, unusual names, snapshot ownership, cleanup, and bounded command lengths.

The Linux public-install check serves metadata read from the actual packed archives through a temporary loopback registry inside its download container. This avoids npm's file-spec manifest extraction of every archive before the real install. npm still verifies each archive's SHA-512 and installs the same packed bytes into an empty consumer. Other dependency metadata redirects to the public npm registry; public tarball URLs remain unchanged. The temporary registry closes before the existing offline lifecycle rebuild and runtime probes. No lifecycle, network-isolation, lock-preservation or provider checks are skipped. Native ARM and Mac qualification keep their existing file installs.