Skip to content

fix(reader): remove unsupported Review Pack and secure build cache - #74

Merged
sarthakagrawal927 merged 2 commits into
mainfrom
agent/reader70-reviewpack-removal-20261005
Oct 4, 2026
Merged

sarthakagrawal927 merged 2 commits into
mainfrom
agent/reader70-reviewpack-removal-20261005

Conversation

@sarthakagrawal927

@sarthakagrawal927 sarthakagrawal927 commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

Summary

Remove the unsupported Review Pack banner, its dead early-access CTA, and its unused component from the populated library. Existing library navigation, controls, and article handling are unchanged.

This scoped change addresses #70 following the owner’s removal decision. It does not create a replacement offering or capture flow. Footer work remains out of scope.

The landing build also used http-cache-semantics@4.2.0, affected by GHSA-ch52-4w7c-c8xp. The advisory has no upstream patched release, and published 4.3.0 still needs the reviewed behavior fix. This PR pins the existing build dependency to 4.3.0 with the small local patch from the reviewed PostTrainLLM fix, and tests the package as resolved through Astro. No runtime dependency was added and the dependency audit remains active.

Validation

  • Fleet Workspace frozen install — passed; lockfile and patch metadata validated
  • Cache semantics regression suite — 27/27 passed through landing Astro's resolved package, including cookie/public/immutable compatibility and serialized policies
  • pnpm quality — format, lint, typecheck, app coverage tests (176), extension typecheck/tests (20), code-health checks, dependency security tests (27), audit (0 critical/high/unexpected), suppressions all passed; the docs check first exposed a path-with-spaces bug, corrected using fileURLToPath, then pnpm docs:check passed (53 Markdown files)
  • pnpm cf:build — passed
  • Extension production build — passed
  • pnpm quality:hygiene and pnpm format:check — passed
  • Push hook lint — 0 errors (38 existing warnings)
  • Exact-head GitHub CI — pending

Authenticated live library baseline showed the banner and current library controls. No private article content was saved to evidence. A signed-in candidate after-render is pending release; the isolated writer has no authenticated fixture.

Candidate: 243a77b (banner removal de9c35b + cache fix 243a77b)

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
reader 243a77b Oct 04 2026, 10:44 PM

@sarthakagrawal927 sarthakagrawal927 changed the title fix(reader): remove unsupported Review Pack banner fix(reader): remove unsupported Review Pack and secure build cache Oct 4, 2026
@sarthakagrawal927
sarthakagrawal927 merged commit 963e0ea into main Oct 4, 2026
3 checks passed
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