Skip to content

feat: change demo placeholder URL from stripe.com to stratechery.com#330

Merged
brendanjryan merged 1 commit into
mainfrom
feat/stratechery-demo-link
Mar 16, 2026
Merged

feat: change demo placeholder URL from stripe.com to stratechery.com#330
brendanjryan merged 1 commit into
mainfrom
feat/stratechery-demo-link

Conversation

@brendanjryan

@brendanjryan brendanjryan commented Mar 16, 2026

Copy link
Copy Markdown
Collaborator

Updates the terminal demo's default URL from stripe.com to stratechery.com/2025/the-agentic-web-and-original-sin/ (Ben Thompson's The Agentic Web and Original Sin).

Changes:

  • terminal-steps.ts — placeholder in article and lookup steps now uses the full Stratechery article URL
  • Terminal.tsx — added stratechery.com to client-side COMPANIES map
  • article.ts — added canned summary; curated summaries now take priority over live Parallel extraction for consistent demo behavior
  • lookup.ts — added canned company entry
  • Unit and e2e tests updated

Types check and all 2675 unit tests pass.

@vercel

vercel Bot commented Mar 16, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
mpp Ready Ready Preview, Comment Mar 16, 2026 11:34pm

Request Review

@chatgpt-codex-connector chatgpt-codex-connector 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0519354528

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +1063 to +1065
if (!hash.startsWith("#")) return;
initialHashHandled.current = true;
const serviceId = hash.slice("#service-".length);
const serviceId = hash.slice(1);

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep backward compatibility for service deep-link hashes

This hash parser now only accepts the new #<serviceId> format, so links generated before this change (for example /services#service-openai, which this component previously wrote) no longer expand the target row because service-openai is treated as a literal ID and fails the services.some(...) check. This breaks existing bookmarked/shared deep links for users who got URLs from the prior behavior.

Useful? React with 👍 / 👎.

Update the article and lookup demo steps to default to stratechery.com.
Add canned summary and company entry for Stratechery.
@brendanjryan brendanjryan force-pushed the feat/stratechery-demo-link branch from 17c0658 to eb40b2d Compare March 16, 2026 23:36
@brendanjryan brendanjryan merged commit f09a07c into main Mar 16, 2026
6 checks passed
@brendanjryan brendanjryan deleted the feat/stratechery-demo-link branch March 16, 2026 23:40
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