Skip to content

Commit 7a6678f

Browse files
committed
ci(ui-merge): the nineteenth Repo Gate joins Lint UI; its ceiling re-derives 30 to 40
#25957 retargeted check:pre-install-import-graph at the root workflows, and it is green on the merged tree, so the step held back for 2d-3 runs now. The F13 rule re-derived with the gates' ~174 s added (558 + 174 = 732 s) gives 40; turbo-cache-save-bound.test.ts pins the new value. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015MoCPtpv6AtTo899tF8Ktq
1 parent c264697 commit 7a6678f

2 files changed

Lines changed: 17 additions & 12 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 16 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -3695,7 +3695,10 @@ jobs:
36953695
# `started_at` from the Actions jobs endpoint, before Build UI Deps existed
36963696
# (so each run includes a cold seven-package build of up to 288 s):
36973697
# - Lint UI: n=42, min 71 s, median 112 s, p95 524 s, max 558 s (9.3 min)
3698-
# => max(27.9, 24.3) => 30, the value it was ported with.
3698+
# => max(27.9, 24.3) => 30, the value it was ported with. Re-derived
3699+
# for objectui's Repo Gates (objectstack#25880), which add ~174 s (their
3700+
# first PR run, 286 s, less the 112 s median): 558 + 174 = 732 s
3701+
# (12.2 min) => max(36.6, 27.2) => 40.
36993702
# - Build & E2E UI: n=51, min 106 s, median 170 s, p95 423 s, max 457 s
37003703
# (7.6 min) => max(22.9, 22.6) => 25, down from objectui's 30 plus the
37013704
# 15 the port added for the in-job build, which no derivation backed.
@@ -3706,7 +3709,7 @@ jobs:
37063709
needs: [filter, ui-deps]
37073710
if: ${{ !cancelled() && needs.filter.outputs.ui_tree != 'false' && needs.filter.outputs.ui != 'false' }}
37083711
runs-on: ubuntu-latest
3709-
timeout-minutes: 30
3712+
timeout-minutes: 40
37103713
permissions:
37113714
contents: read
37123715
env:
@@ -3776,9 +3779,8 @@ jobs:
37763779
pnpm --dir ui run check:bash32-floor
37773780
37783781
# ── objectui's Repo Gates (objectstack#25880) ────────────────────────
3779-
# Eighteen of the nineteen gates of `ui/.github/workflows/repo-gates.yml`
3780-
# (the last is noted below), which GitHub stopped running when the tree
3781-
# moved under `ui/` (it reads only
3782+
# The nineteen gates of `ui/.github/workflows/repo-gates.yml`, which
3783+
# GitHub stopped running when the tree moved under `ui/` (it reads only
37823784
# this repository's `.github/workflows/`). They judge what objectui ships
37833785
# — `ui/content/docs/**`, `ui/skills/**`, the package READMEs and the
37843786
# documented component types — so the change most likely to trip them is
@@ -3803,12 +3805,15 @@ jobs:
38033805
if: ${{ !cancelled() }}
38043806
run: pnpm --dir ui run check:action-ref-convention
38053807

3806-
# Not yet here, the nineteenth: `Pre-Install Import Graph Check`
3807-
# (`check:pre-install-import-graph`). It derives its population from
3808-
# `ui/.github/workflows/`, where ten workflows still run the deleted
3809-
# `scripts/objectstack-pin.mjs`, so it is red on `main` today for that
3810-
# reason alone. It joins this list when `ui/.github/` is retired
3811-
# (objectstack#25712), retargeted to this job.
3808+
# Self-test first: a scan whose recogniser is broken reports a clean
3809+
# tree. It derives its population from this repository's workflows, so
3810+
# it walks this job's install-free steps, itself included.
3811+
- name: Pre-Install Import Graph Check — Check the pre-install gates' import graphs
3812+
if: ${{ !cancelled() }}
3813+
run: |
3814+
pnpm --dir ui run check:pre-install-import-graph --self-test
3815+
pnpm --dir ui run check:pre-install-import-graph
3816+
38123817
- name: Shell Escape Residue Scan — Check for machine-produced shell-escape residue in fenced examples
38133818
if: ${{ !cancelled() }}
38143819
run: pnpm --dir ui run check:shell-escape-residue

‎ui/scripts/__tests__/turbo-cache-save-bound.test.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ const CACHE_JOBS = [DEPS_JOB, ...Object.values(UI_JOBS)];
7373
const CEILINGS: Record<string, number> = {
7474
[DEPS_JOB]: 25,
7575
[UI_JOBS.typeCheck]: 40,
76-
[UI_JOBS.lint]: 30,
76+
[UI_JOBS.lint]: 40,
7777
[UI_JOBS.buildE2e]: 25,
7878
};
7979

0 commit comments

Comments
 (0)