Repository navigation
Triage: open issue/PR overlaps, consolidation candidates, and merge order (2026-09-28) #783
Description
Activity
Progress update on the "P0 fixes + #672 CI prerequisite" scope of this triage:
- fix(security): reject 4-space/tab-indented fence openers as fences #785 — fix(security): reject 4-space/tab-indented fence openers as fences (closes [scanner] MDX active-content gate bypass: 4-space/tab-indented fence opener blanks live script content past findActiveContent #689)
- fix: index docs at the route docs are actually served on #786 — fix: index docs at the route docs are actually served on (closes [quality] site search indexes only the blog: docsRouteBasePath default does not match docs routeBasePath #769)
- ci: run the Playwright e2e suite on every pull request #787 — ci: run the Playwright e2e suite on every pull request (closes [quality] Playwright e2e suite is never run in CI — tests/e2e/smoke.spec.js has no workflow #672)
All three are green on
Validate repository/Lint repository/CodeQL, and #787 adds and passes the newEnd-to-end testsjob (16/16 specs). None of these PRs touch anyhold-labeled item; per GOVERNANCE.md they're posted for maintainer review, not self-merged.Remaining items from the priority/merge-order list (#755's actual workflow-ordering fix, #721, the documentation cluster, #754's narrowing, #700's re-verification, #46, #156) are still open and unstarted.
hivecommons-hive commented
on Sep 28, 2026 ContributorMore actionsLive re-verification of the triage state (main @e0b7571)
Confirmed against current main and the PR/issue states:
- Point 1 (Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754 vs fix(security): gate remote resource references in imported SVG diagrams #775): fix(security): gate remote resource references in imported SVG diagrams #775 is now MERGED (2026-09-28T15:28Z), so the contingency landed — posted a narrowing comment on Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754; only the dynamic
static/subdirectory discovery goal remains there. - Point 2 (Automation workflows must validate and format what they generate before committing #755 vs test: guard regenerate-format-validate-test ordering for data-generating automation jobs #758): test: guard regenerate-format-validate-test ordering for data-generating automation jobs #758 is now MERGED with
Refs #755(correct, non-closing). Verified: the new contract suite passes 8/8 with all 7 violations still recorded inKNOWN_ORDERING_VIOLATIONS— the workflow reorder is still unwritten and Automation workflows must validate and format what they generate before committing #755 stays open for it. (One nuance vs the triage text: merging test: guard regenerate-format-validate-test ordering for data-generating automation jobs #758 did not turn CI red, since the violations are allow-listed; instead the staleness guard turns CI red only when a workflow is fixed but its entry isn't deleted.) - [scanner] MDX active-content gate bypass: 4-space/tab-indented fence opener blanks live script content past findActiveContent #689 is CLOSED via merged fix(security): reject 4-space/tab-indented fence openers as fences #785 (indented-fence gate), matching the P0 list in the progress comment above.
- Point 3 ([ci-maintainer] Automated import PRs run no CI: GITHUB_TOKEN-created PRs do not trigger pull_request workflows #721 vs chore: refresh community profiles #763/chore: import latest reference architectures #681): still live — both automation PRs remain in the actionable queue with
mergeable_state=blockedand no CI checks beyond DCO.
scanner agent (ACMM L4 — issues-only mode)
🐝 Hive Agent:
scanner| Instance:hosted-available-lke648397-260827-5n31| SHA:unknown— hive: agent=scanner backend=copilot model=kimi-k3 copilot=1.0.88
- Point 1 (Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754 vs fix(security): gate remote resource references in imported SVG diagrams #775): fix(security): gate remote resource references in imported SVG diagrams #775 is now MERGED (2026-09-28T15:28Z), so the contingency landed — posted a narrowing comment on Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754; only the dynamic
hivecommons-hive commented
on Sep 28, 2026 ContributorMore actionsState update — triage points overtaken by today's merges (main @50c4907)
Verified live 2026-09-28 ~20:05Z. Three of the triage's points are now stale:
- Point 1 (Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754 vs fix(security): gate remote resource references in imported SVG diagrams #775): fully resolved — Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754 is CLOSED. PR fix(security): gate remote resource references in imported SVG diagrams #775 shipped the external-URL SVG gate and PR fix(security): fail loudly on an ungated static/ subdirectory #803 (merged 18:45Z, "Fixes Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754") shipped the remaining goal:
checkForUngatedStaticDirs()walks top-levelstatic/and hard-fails on any ungated subdirectory, withexemptTopLevelDirs(fonts/) as the explicit escape. Nothing left to narrow. - Point 2 (Automation workflows must validate and format what they generate before committing #755 vs test: guard regenerate-format-validate-test ordering for data-generating automation jobs #758): progressed — PR fix: format generated architecture data before opening a PR #795 (18:21Z) added
fix:formatto import-architectures.yml and deleted the corresponding entry fromKNOWN_ORDERING_VIOLATIONS; PR fix: skip refresh-community-people write/PR when nothing changed #793 (16:55Z) skips the people-refresh write when nothing changed. Remaining violations in the contract test: 6 entries (unit-order ×2, format ×2, people validate, deploy gate coverage). Automation workflows must validate and format what they generate before committing #755 still OPEN and correctly so. - Point 3 ([ci-maintainer] Automated import PRs run no CI: GITHUB_TOKEN-created PRs do not trigger pull_request workflows #721 vs chore: refresh community profiles #763/chore: import latest reference architectures #681): resolved — chore: import latest reference architectures #681 merged 17:13Z (import PR, now runs CI; import workflow also made on-demand instead of scheduled by ci: make reference architecture import on-demand instead of scheduled #791 at 16:41Z), chore: refresh community profiles #763 closed. [scanner] Automation PRs never run CI — GITHUB_TOKEN-created PRs don't trigger pull_request workflows, leaving PR #681 permanently blocked #715 (the tracking issue for automation-PR CI) is CLOSED.
Also merged since this triage: #790 (redirect-stub tests), #794 (confine images to import-gate-allowed hosts), #796–#800 (docs).
State check by scanner agent (ACMM L4 — issues-only mode)
🐝 Hive Agent:
scanner| Instance:hosted-available-lke648397-260827-5n31| SHA:50c4907— hive: agent=scanner backend=copilot model=kimi-k3 copilot=1.0.88
- Point 1 (Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754 vs fix(security): gate remote resource references in imported SVG diagrams #775): fully resolved — Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754 is CLOSED. PR fix(security): gate remote resource references in imported SVG diagrams #775 shipped the external-URL SVG gate and PR fix(security): fail loudly on an ungated static/ subdirectory #803 (merged 18:45Z, "Fixes Strengthen the static/ asset security gate: discover directories dynamically and block external references in SVGs #754") shipped the remaining goal:
Purpose
A pass over the current 23 open issues and 13 open PRs in cncf/endusers to flag likely duplicates/overlaps, propose consolidations, and set a priority and merge order. All PR numbers/CI states below were verified live via
gh pr view/gh issue viewat time of writing. Nohold-labeled item was merged or modified as part of this triage (perGOVERNANCE.md).Overlaps and duplicate candidates
static/subdirectories) or split into a new, smaller issue so it isn't left describing already-shipped work.refresh-community-people, chore: import latest reference architectures #681import-architectures) are currentlymergeStateStatus: BLOCKEDbecause branch protection is waiting on aValidate repositorycheck that GITHUB_TOKEN-authored PRs never trigger. These two PRs are live evidence for [ci-maintainer] Automated import PRs run no CI: GITHUB_TOKEN-created PRs do not trigger pull_request workflows #721 and are stuck until it's fixed (or a maintainer manually re-runs/approves).Consolidation candidates
CONTRIBUTING.md/AGENTS.md(stale validator claims, missing e2e mention, wrong hosting domain, wrong coverage command). None has a PR yet. Recommend one combined "docs accuracy sweep" PR/issue instead of four separate small diffs to the same two files — reduces review overhead and merge-conflict risk from touchingCONTRIBUTING.mdfour times in parallel.CLEAN, so they don't need to be merged as one PR, but they should be tracked as one initiative and gated on [quality] Playwright e2e suite is never run in CI — tests/e2e/smoke.spec.js has no workflow #672 (see below) rather than reviewed as eight unrelated PRs.Priority triage
P0 — correctness/security, blocking, no PR yet
BLOCKED(two live PRs affected today)P1 — ready to merge (green CI,
hold-gated, awaiting maintainer review)P1 — infra prerequisite for the e2e initiative, no PR yet
P2 — ready to merge (green CI,
hold-gated), gate on #672 landing firstP2 — automation correctness, partial fix only
P2 — blocked automated data PRs, unblocks once #721 is fixed
P3 — lower-risk cleanup, no PR yet
P4 — non-technical / long-horizon
docs/community/members.mdheading is still "Member Directory") but old enough to need a fresh confirmation it's still wanted before scopingSuggested merge order
Notes on method
Derived from
gh issue list --repo cncf/endusers --state open(23 issues),gh pr list --repo cncf/endusers --state open(13 PRs, including #782 which is very recent), each PR'smergeStateStatus/statusCheckRollup, and cross-referencing issue/PR bodies for explicit "Closes #NNN" links and consolidation notes. Nohold-labeled issue or PR was modified.