diff --git a/CHANGELOG.md b/CHANGELOG.md index df7103e..9c99d9c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -286,6 +286,15 @@ Use this file to track chronology, not release notes. Keep entries short, factua - Blockers: I did not run a true WSL shell here, so the WSL fix is validated by the deterministic source patch plus tests rather than an actual Windows-hosted browser-launch repro. - Next: Push the WSL/login pass and close the stale issues and PRs that are already superseded by `main`. +### 2026-04-14 09:35 PDT — review-findings-and-audit-cleanup + +- Objective: Fix the remaining concrete issues found in the deeper review pass instead of stopping at tracker cleanup. +- Changed: Updated the `pi-web-access` patch so Feynman defaults search workflow to `none` without disabling explicit `summary-review`; softened the research workflow prompts so only unattended/one-shot runs auto-continue while interactive users still get a chance to request plan changes; corrected uninstall docs to mention `~/.ahub` alongside `~/.feynman`; bumped the root `basic-ftp` override from `5.2.1` to `5.2.2`. +- Verified: Ran `npm test`, `npm run typecheck`, `npm run build`, `cd website && npm run build`, and `npm audit`; root audit is now clean. +- Failed / learned: Astro still emits a duplicate-content-id warning for `website/src/content/docs/getting-started/installation.md`, but the website build succeeds and I did not identify a low-risk repo-side fix for that warning in this pass. +- Blockers: The duplicate-id warning remains as a build warning only, not a failing correctness gate. +- Next: If desired, isolate the Astro duplicate-id warning separately with a minimal reproduction rather than mixing it into runtime/CLI maintenance. + ### 2026-04-12 13:20 PDT — capital-france (citation verification brief) - Objective: Verify citations in the capital-of-France draft and produce a cited verifier brief. diff --git a/package-lock.json b/package-lock.json index 857616e..7352330 100644 --- a/package-lock.json +++ b/package-lock.json @@ -2530,9 +2530,9 @@ "license": "MIT" }, "node_modules/basic-ftp": { - "version": "5.2.1", - "resolved": "https://registry.npmjs.org/basic-ftp/-/basic-ftp-5.2.1.tgz", - "integrity": "sha512-0yaL8JdxTknKDILitVpfYfV2Ob6yb3udX/hK97M7I3jOeznBNxQPtVvTUtnhUkyHlxFWyr5Lvknmgzoc7jf+1Q==", + "version": "5.2.2", + "resolved": "https://registry.npmjs.org/basic-ftp/-/basic-ftp-5.2.2.tgz", + "integrity": "sha512-1tDrzKsdCg70WGvbFss/ulVAxupNauGnOlgpyjKzeQxzyllBLS0CGLV7tjIXTK3ZQA9/FBEm9qyFFN1bciA6pw==", "license": "MIT", "engines": { "node": ">=10.0.0" diff --git a/package.json b/package.json index e497b3d..4894b0f 100644 --- a/package.json +++ b/package.json @@ -66,7 +66,7 @@ "dotenv": "^17.3.1" }, "overrides": { - "basic-ftp": "5.2.1", + "basic-ftp": "5.2.2", "@modelcontextprotocol/sdk": { "@hono/node-server": "1.19.13", "hono": "4.12.12" @@ -79,7 +79,7 @@ "proxy-agent": { "pac-proxy-agent": { "get-uri": { - "basic-ftp": "5.2.1" + "basic-ftp": "5.2.2" } } },