Skip to content

feat: route SignalOps home CTA to replay [agent:clawd]#9

Merged
TargiX merged 1 commit into
mainfrom
clawd/signalops-home-replay-cta
Jul 10, 2026
Merged

feat: route SignalOps home CTA to replay [agent:clawd]#9
TargiX merged 1 commit into
mainfrom
clawd/signalops-home-replay-cta

Conversation

@TargiX

@TargiX TargiX commented Jul 8, 2026

Copy link
Copy Markdown
Owner

agent:clawd

Summary

  • Route the SignalOps homepage secondary CTA directly into the guided replay deep link (/cockpit?replay=alibaba-p95&step=0) instead of the unframed incident detail page.
  • Document the fastest shareable replay URL in README and route list.

Why now

The replay deep-linking work is already merged, but the product homepage still sent reviewers to the incident detail route. This makes the first portfolio click land on the strongest guided demo path.

Verification

  • pnpm --config.dangerously-allow-all-builds=true run typecheck
  • pnpm --config.dangerously-allow-all-builds=true run lint
  • pnpm --config.dangerously-allow-all-builds=true run build
  • Static build smoke: generated homepage contains href="/cockpit?replay=alibaba-p95&step=0"
  • Codex review: no findings; verified /cockpit?replay=alibaba-p95&step=0 returned 200 and query maps to an existing replay scenario.

Notes

  • claude CLI was not available in this cron environment (command not found), so Codex was used for the second-pass review.
  • Ilya approves all merges manually.

Summary by CodeRabbit

  • New Features

    • Updated the main hero call-to-action to open a guided replay experience directly.
    • Added a clearer guided replay route to the walkthrough and route list, making it easier to share and start from the first step.
  • Copy Updates

    • Changed the CTA label to “Watch guided replay” for better clarity.

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
signalops Ready Ready Preview, Comment Jul 8, 2026 3:40am

@coderabbitai

coderabbitai Bot commented Jul 8, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: da3eac96-eda8-40aa-be9a-1f04fbf183e2

📥 Commits

Reviewing files that changed from the base of the PR and between c47f846 and df68fb6.

📒 Files selected for processing (2)
  • README.md
  • src/components/product-home.tsx

📝 Walkthrough

Walkthrough

The hero CTA link in product-home.tsx was retargeted from an incidents detail page to a cockpit guided replay URL, with its label changed to "Watch guided replay." README.md was updated to document this new entry point and route.

Changes

Guided replay entry point update

Layer / File(s) Summary
Hero CTA link and label update
src/components/product-home.tsx
Secondary hero replay link now points to /cockpit?replay=alibaba-p95&step=0 instead of /incidents/inc_411, and label changed to "Watch guided replay".
README documentation update
README.md
Adds a sentence noting the fastest shareable entry URL and a new bullet under Routes documenting the guided replay starting point.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Possibly related PRs

  • TargiX/Signalops#5: Introduced the guided incident replay feature and /cockpit?replay=... routing that this PR's links now target.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: routing the SignalOps homepage CTA to the guided replay flow.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch clawd/signalops-home-replay-cta

Comment @coderabbitai help to get the list of available commands.

@TargiX
TargiX merged commit 6e49173 into main Jul 10, 2026
4 checks passed
@TargiX
TargiX deleted the clawd/signalops-home-replay-cta branch July 10, 2026 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant