Skip to content

test(seo): lock APISIX documentation ownership signals - #2119

Merged
Yilialinn merged 1 commit into
masterfrom
codex/stabilize-doc-search-signals
Sep 14, 2026
Merged

Yilialinn merged 1 commit into
masterfrom
codex/stabilize-doc-search-signals

Conversation

@Yilialinn

Copy link
Copy Markdown
Contributor

Summary

  • lock API7-owned APISIX documentation as hosted copies with cross-site canonicals
  • preserve Chinese canonical targets on docs.apiseven.com
  • keep cross-site canonical pages out of APISIX hreflang and sitemaps
  • protect APISIX-owned current docs as indexable, self-canonical sitemap entries
  • verify /docs/apisix/ and historical Ingress redirects remain one hop
  • behavior-test historical and next documentation noindex/canonical policy

No reproducible production signal conflict was found, so this PR does not add speculative redirects or change the existing API7 acquisition strategy.

Validation

  • npm run sync (0 errors; existing content warnings only)
  • npm run test:redirects
  • npm run test:sitemap
  • JavaScript syntax checks
  • API7-owned discovery check: 166 EN/ZH pages, no duplicate paths
  • ESLint for the changed TSX file
  • git diff --check
  • independent review: 0 P0, 0 P1, 0 P2 findings

The full Astro build, post-build ownership audit, and desktop/mobile Playwright suite run in PR CI.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No unresolved review issues were identified.

Pull request overview

Adds automated SEO ownership checks for APISIX documentation across Astro and Docusaurus.

Changes:

  • Adds canonical, robots, hreflang, sitemap, redirect, and historical-documentation tests.
  • Extracts reusable versioned-document SEO logic.
  • Updates dependencies and CI validation.
File summaries
File Description
next/tests/fixtures/documentation-search-signals.mjs Shared SEO fixtures and document discovery
next/tests/e2e/seo-signals.spec.mjs Browser-level SEO assertions
next/scripts/generate-sitemaps.test.mjs Sitemap ownership coverage
next/scripts/check-ingress-redirects.mjs Redirect validation
next/scripts/check-historical-doc-signals.mjs Historical documentation signal tests
next/scripts/check-documentation-ownership-signals.mjs Built-site ownership audit
next/package.json Test script and dependency updates
next/package-lock.json Locked dependency updates
doc/src/theme/LayoutHead/versionedDocSignals.mjs Shared versioned-document signal logic
doc/src/theme/LayoutHead/versionedDocSignals.d.mts Signal logic type declarations
doc/src/theme/LayoutHead/index.tsx Uses shared SEO signal logic
.github/workflows/lint.yml CI synchronization and validation
Review details

Files not reviewed (1)

  • next/package-lock.json: Generated file
  • Files reviewed: 11/12 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Yilialinn Yilialinn self-assigned this Sep 14, 2026
@Yilialinn
Yilialinn merged commit 5e7af42 into master Sep 14, 2026
8 checks passed
@Yilialinn
Yilialinn deleted the codex/stabilize-doc-search-signals branch September 14, 2026 06:06
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.

4 participants