Skip to content

fix(eng-analytics): repair stale detail-page links - #72002

Merged
rnegron merged 2 commits into
masterfrom
worktree-eng-analytics-alerter-links
Jul 17, 2026
Merged

fix(eng-analytics): repair stale detail-page links#72002
rnegron merged 2 commits into
masterfrom
worktree-eng-analytics-alerter-links

Conversation

@rnegron

@rnegron rnegron commented Jul 17, 2026

Copy link
Copy Markdown
Member

Problem

Master-red Slack alerts link each failing workflow to its eng-analytics run-history page. Those links 404 — the redesign (#69516) moved detail pages under a repos/ prefix, but two link builders outside the frontend weren't updated.

Changes

  • Alerter runsUrlFor(): add repos/ prefix.
  • Seed command demo-PR link: add repos/, pull/ -> pull-requests/, reuse SEED_REPOSITORY.
  • Updated the two alerter URL test assertions (38 pass).

Note

Only fixes new alerts; the already-posted broken message won't self-heal.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

I (Claude) traced the broken link to the repos/ route change and swept for other stale builders. Frontend routes were already correct.

@rnegron rnegron added the skip-agent-review Save $$$, skip auto agent reviews (Greptile) — use for trivial or chore PRs label Jul 17, 2026
@rnegron rnegron self-assigned this Jul 17, 2026
@rnegron
rnegron marked this pull request as ready for review July 17, 2026 15:13
@rnegron rnegron added the stamphog Request AI approval (no full review) label Jul 17, 2026
@rnegron
rnegron enabled auto-merge (squash) July 17, 2026 15:13
@rnegron
rnegron requested a review from a team July 17, 2026 15:13

@stamphog stamphog Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small, contained fix to two stale URL builders (Slack alert links and a seed command's demo-PR message) with matching test updates; author is on the owning team and the change carries no risk to CI execution, data models, or security.

  • Author wrote 0% of the modified lines and has 39 merged PRs in these paths (familiarity MODERATE).
  • 👍 on the PR from hex-security-app[bot].
Gate mechanics and policy version
Gate Result
prerequisites all clear
deny-list no deny categories matched
size 6L, 2F substantive, 10L/3F incl. docs/generated/snapshots — within ceiling
tier T1-agent / T1b-small (10L, 3F, two-areas, fix)
stamphog 2.0.0b3 .stamphog/policy.yml @ 0167f1c · reviewed head 3f315ec

@trunk-io

trunk-io Bot commented Jul 17, 2026

Copy link
Copy Markdown

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@rnegron
rnegron merged commit 163a687 into master Jul 17, 2026
214 of 222 checks passed
@rnegron
rnegron deleted the worktree-eng-analytics-alerter-links branch July 17, 2026 15:37
@deployment-status-posthog

deployment-status-posthog Bot commented Jul 17, 2026

Copy link
Copy Markdown

Deploy status

Environment Status Deployed At Workflow
dev ✅ Deployed 2026-07-17 16:01 UTC Run
prod-us ✅ Deployed 2026-07-17 16:21 UTC Run
prod-eu ✅ Deployed 2026-07-17 16:19 UTC Run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-agent-review Save $$$, skip auto agent reviews (Greptile) — use for trivial or chore PRs stamphog Request AI approval (no full review)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant