Skip to content

Anchor the console's pasted-link host check (LIVE-1272) - #3806

Merged
hellofrequencylab merged 1 commit into
mainfrom
claude/placard-link-anchor
Oct 22, 2026
Merged

hellofrequencylab merged 1 commit into
mainfrom
claude/placard-link-anchor

Conversation

@hellofrequencylab

Copy link
Copy Markdown
Owner

What

A one-line follow-up to #3805: the CyberUnicorn console's check for whether you pasted a frequencylocal.com link now anchors the end of the host name.

Why

CodeQL flagged the unanchored pattern as a high alert (code scanning alert 359). The risk was low. The pattern only decides in the browser whether the box holds a link or a search, and startPageTag already compares the parsed hostname on the server. The alert would stay open on main if nothing changed it.

How to review (owner)

  1. Wait for CI and CodeQL to go green.
  2. Nothing to click: what you see on screen does not change.

How / files

  • components/spaces/loyalty/cyberunicorn-console.tsx: frequencylocal\.com now becomes frequencylocal\.com(\/|$).
  • docs/ledger/rows/LIVE-1272.json: the row's detail notes the follow-up.

Verification

  • eslint on the file
  • scripts/probes/cu5k-tag-any-page.mjs passes

Notes / follow-ups

None.


Generated by Claude Code

CodeQL flagged the unanchored frequencylocal.com pattern. It only picks between
search and paste in the browser; startPageTag compares the parsed hostname.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015Aw32AU4vVqCSEoNA5aP3U
@vercel

vercel Bot commented Oct 22, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
frequency-web Ready Ready Preview Oct 22, 2026 1:12am UTC

Request Review

Copy link
Copy Markdown
Owner Author

test failed on one case, scripts/backlog-contract.test.ts > "the full --probes=all sweep is green, and stays inside its cost budget". It hit its 90 s timeout; every other case passed (23,796 tests). This PR changes one regex in the console and one row's detail text, and adds no probe. The same timeout failed once on #3794 and passed when that job was re-run. No fix exists yet for the sweep's runtime on CI runners, so I re-ran the failed job once.


Generated by Claude Code

@hellofrequencylab
hellofrequencylab merged commit 539e451 into main Oct 22, 2026
17 of 18 checks passed

This branch was successfully deployed

1 active deployment
Preview — a3206f13 Deployed Oct 22, 2026 by vercel[bot]
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.

2 participants