Skip to content

feat(special): add Special:AncientPages report page#1956

Closed
Martin-Blau wants to merge 1 commit into
e35ventura:testfrom
Martin-Blau:feat/ancientpages-report-page
Closed

feat(special): add Special:AncientPages report page#1956
Martin-Blau wants to merge 1 commit into
e35ventura:testfrom
Martin-Blau:feat/ancientpages-report-page

Conversation

@Martin-Blau

Copy link
Copy Markdown
Contributor

Summary

Add Special:AncientPages as a new report page plus its JSON companion endpoint.

This PR adds:

  • /wiki/special/ancientpages/ for a human-readable oldest-pages report
  • /wiki/special/ancientpages.json for machine-readable output
  • shared build-time ranking logic based on article history
  • sitemap and share-preview wiring so the new route behaves like the existing Special pages
  • a build check that verifies the ranking against generated article history

Why This Should Ship

Taopedia already exposes several maintenance-oriented Special pages, but it does not currently expose an oldest-pages report. Special:AncientPages fills that gap with a route that is useful for exploration, historical context, and future maintenance work.

This is a focused feat(special) addition rather than a broad UI change:

  • it uses existing article-history data already generated at build time
  • it follows the same route family as the accepted/safe Special-page work in this repo
  • it adds both HTML and JSON surfaces, matching existing Taopedia patterns
  • it does not duplicate an existing Special page or an already-open PR

Review Notes

Affected routes:

  • /wiki/special/ancientpages/
  • /wiki/special/ancientpages.json
  • /og/special/ancientpages.png

Validation:

  • npm run build
  • included postbuild checks now pass, including node scripts/check-ancient-pages.js

Behavior proof captured on July 1, 2026:

  • live route before this PR: https://taopedia.org/wiki/special/ancientpages/ returns the site 404 page
  • built route after this PR: /wiki/special/ancientpages/ renders the new report page
  • screenshot viewport: 1440px wide

Before:

Before: live Special AncientPages 404

After:

After: built Special AncientPages report

Target Branch

  • This PR targets test
  • This PR targets main only because it promotes test to production

Security Checklist

  • No secrets, tokens, build hooks, wallet seeds, or private endpoints are included
  • No unnecessary workflow, dependency, or deployment changes

@Martin-Blau Martin-Blau requested a review from e35ventura as a code owner July 1, 2026 20:07
@taopedia-site-maintainer taopedia-site-maintainer Bot added feature Website feature or behavior request ui-ux Visual design, layout, or usability work labels Jul 1, 2026
@taopedia-site-maintainer

Copy link
Copy Markdown

Verdict: Needs visual approval

PR #1956: feat(special): add Special:AncientPages report page

Engineering:

  • Review type: Visual/product review
  • CI: All required CI checks passed: build
  • Cross-repo build: pass
  • Risk: high
  • Changed files: 9
  • Changed lines: 235

Changed files:

  • package.json
  • scripts/ancient-pages.js
  • scripts/check-ancient-pages.js
  • scripts/check-sitemap.js
  • src/lib/ancient-pages-context.ts
  • src/lib/share-preview.ts
  • src/pages/sitemap.xml.ts
  • src/pages/wiki/special/ancientpages.astro
  • src/pages/wiki/special/ancientpages.json.ts

Guardrails:

  • Operational/build/deploy files changed; maintainer review required.
  • Visual layout/style/rendering files changed; visual approval required.

Decision surface:

  • Checks and cross-repo build passed; this is waiting on visual approval only.
  • If the after screenshots look correct, comment /merge.
  • If they look wrong or the change should not ship, comment /close.

What to check:

  • Special page /wiki/special/ancientpages/: page title, table/list content, navigation links, and empty/error states.
  • Changed page route(s): src/pages/sitemap.xml.ts.

Preview links:

  • Production reference: https://taopedia.org
  • No automatic Netlify preview is available. Comment /preview to create on-demand before/after draft deploys for this PR.

Cross-repo validation:

  • pass merged checkout: Validated site PR head 9acdc44 merged into test, with articles test.
  • pass npm ci: Dependencies installed.
  • pass cross-repo build: GitHub Actions required build check passed; local build not run to completion on the constrained maintainer host. Original local detail: ...(truncated)
    �[22m �[34m[vite]�[39m �[32m✓ built in 43.08s�[39m
    �[2m20:17:25�[22m �[34m[vite]�[39m �[32m✓ built in 230ms�[39m
    �[2m20:17:25�[22m �[34m[build]�[39m Rearranging server assets...

�[42m�[30m generating static routes �[39m�[49m
�[2m20:17:38�[22m �[34m├─�[39m �[2m/.well-known/security.txt�[22m �[2m(+42ms)�[22m
�[2m20:17:38�[22m �[34m├─�[39m �[2m/404.html�[22m �[2m(+13ms)�[22m
�[2m20:17:38�[22m �[34m├─�[39m �[2m/atom.xml�[22m �[2m(+32ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/feed.json�[22m �[2m(+29ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/feeds.opml�[22m �[2m(+4ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/og/category/Consensus.png�[22m �[2m(+424ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/og/category/Core_Concepts.png�[22m �[2m(+157ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/og/category/Correction.png�[22m �[2m(+121ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/og/category/Cryptography.png�[22m �[2m(+110ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/og/category/Delegation.png�[22m �[2m(+110ms)�[22m
�[2m20:17:39�[22m �[34m├─�[39m �[2m/og/category/Development.png�[22m �[2m(+83ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Diagnostics.png�[22m �[2m(+82ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Economics.png�[22m �[2m(+92ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Emission.png�[22m �[2m(+92ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Emissions.png�[22m �[2m(+101ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Errors.png�[22m �[2m(+94ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Events.png�[22m �[2m(+66ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/EVM.png�[22m �[2m(+90ms)�[22m
�[2m20:17:40�[22m �[34m├─�[39m �[2m/og/category/Governance.png�[22m

Human decision:

  • Review only the visual evidence shown above for product fit.
  • Comment /merge to merge if this should ship.
  • Comment /close to close if this should not ship.

Automated review by Taopedia Site Maintainer.

@e35ventura

Copy link
Copy Markdown
Owner

/close this should be based on last edit

@taopedia-site-maintainer

Copy link
Copy Markdown

Closing by maintainer command /close.


Automated review by Taopedia Site Maintainer.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Website feature or behavior request ui-ux Visual design, layout, or usability work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants