Skip to content

fix(stats): remove obsolete FAQ rich-result metadata - #1436

Merged
tannerlinsley merged 1 commit into
mainfrom
taren/npm-stats-structured-data
Oct 26, 2026
Merged

tannerlinsley merged 1 commit into
mainfrom
taren/npm-stats-structured-data

Conversation

@tannerlinsley

@tannerlinsley tannerlinsley commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Remove the duplicate FAQ JSON-LD from NPM Stats. The visible FAQ remains, along with the WebApplication metadata and its clean /stats/npm URL.

This completes the FAQ metadata review requested in #1152. Google retired FAQ rich results in May 2026, and no other consumer is documented in this codebase. The broader issue stays open for deployment and comparison-route verification.

AI review notes:
The actual local application returns HTTP 200 with the visible FAQ and WebApplication JSON-LD intact, and no FAQPage script. Its canonical and WebApplication URL both point to https://tanstack.com/stats/npm.
Full pnpm test passed locally. This metadata change has not been deployed.

Summary by CodeRabbit

  • Changes
    • Removed FAQ structured data from the npm statistics page. The visible FAQ section is unchanged.

@changeset-bot

changeset-bot Bot commented Oct 26, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 79b6f89

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 840da5c4-895d-4ae9-b2b3-4216cd33cd02
Commits

Reviewing files that changed from the base of the PR and between 66c846f and 79b6f89.

Files selected for processing: 1
  • src/routes/stats/npm/index.tsx
Files with no reviewable changes: 1
  • src/routes/stats/npm/index.tsx

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 0 remain after this review.

Walkthrough

The NPM statistics route no longer defines or emits FAQPage structured data. The visible FAQ section and WebApplication JSON-LD remain.

Changes: NPM statistics metadata

Layer / File(s) Summary
Remove FAQPage structured data
src/routes/stats/npm/index.tsx
The route removes the FAQPage data object and its JSON-LD script. The visible FAQ section and WebApplication JSON-LD remain.

Priority: Low

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

Change: Bug fix

Possibly related PRs

Merge Risk: Minimal · up to 79b6f

The metadata removal appears ready for normal deployment checks; no merge-blocking issue was identified.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: removing obsolete FAQ rich-result metadata from the NPM stats route.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@tannerlinsley
tannerlinsley merged commit e8a9bb6 into main Oct 26, 2026
8 checks passed
@tannerlinsley
tannerlinsley deleted the taren/npm-stats-structured-data branch October 26, 2026 20:29
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