memory: log PR #11 review + merge

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: devanshutak25

memory/decisions.md

@@ -9,6 +9,10 @@ Architectural / structural decisions w/ rationale + date.
   - Impact:
 
 ## Log
+- **2026-08-21 - Merged PR #11 (Thridy -> §01 icons).**
+  - Decision: user said "merge PRs, do all necessary checks first". Only open PR was #11 (kiranjinnil, self-disclosed author of Thridy). Merged with a real `--no-ff` merge commit (not squash) so GitHub auto-closes the PR - the 2026-08-04 squash of PR #10 left it open because squash-merge does not close a PR without `gh`.
+  - Checks: closed enums valid + matches §01 asset-source-website convention (`platform: web`); readme_tags 2/2; chunk 29/50; catalog dupe grep `thridy` = 1 hit; URL 200 + license page confirms free/no-signup/no-attribution; style clean. CI never ran on the PR (0 check-runs, `mergeable_state: unstable`, first-time-contributor approval gate), so ran CI parity locally: validate ✓ 428 warnings (= main baseline, 0 new), render.js exit 0, export-data.js exit 0.
+  - Impact: main d98c067 pushed; open PRs 0. README.md NOT regenerated (separate step, user's call). Tooling note: `gh` CLI still absent - PR listing/inspection done via the public GitHub REST API from PowerShell, diff via `git fetch origin pull/N/head`.
 - **2026-06-30 — Marked 21 bot-block/transient link-check false positives `unreachable` (un-deprecated).**
   - Decision: after the sweep + report-only, user said "mark unreachable for now." Downgraded all 21 not-genuinely-dead `broken` entries → `url_status: unreachable` + removed `deprecated: true` (conservative holding status; un-hides live tools without asserting `ok`).
   - Scope: 18 confirmed-alive (3ds Max, MotionBuilder, VRED, Inventor, ReCap, RV, Flow Capture, Flow Production Tracking, Mo-Sys StarTracker, Celestia, Domestika, Wonder Studio, HDR Shop, bsky.art, Fast Palette, IEEE, Field3D, 3D Maths Cheat Sheet) + 3 transient (Brandfetch 403, Houdini Blueprints 502, Importance Sampling 444). Excluded Mudbox (legacy-defunct, deprecated intentional) + the genuinely-dead links (stay broken+deprecated).

memory/user-prompts.md

@@ -758,3 +758,10 @@ Every user prompt/question, verbatim, w/ timestamp + short answer note.
 > and check links freshness
 
 **Answered:** Added MipMap (drone mapping photogrammetry; mining/construction/surveying; textured mesh + point cloud + Gaussian splats; from $29.90/mo, perpetual option) to `10-tools-pipeline/photogrammetry-scanning/03` (31 -> 32), beside Pix4D. User picked placement (§10 only, no §12 mirror), platform `[win, mac]` (mirrors Pix4D; site is JS-rendered so OS unconfirmed), tech `[photogrammetry, gaussian-splatting]` (no freeform `lidar`). Link freshness: catalog was last verified 2026-08-01 with 65 broken / 59 unreachable untriaged; user chose full rescan writing all chunks. `node scripts/check-links.js` over 3,543 URLs -> OK 3,426, broken 58, unreachable 59, redirects 0. 180 files touched, 0 NEW `deprecated: true` (all 58 broken were already flagged). Report: `_maintenance/link-check-2026-08-18.md`. Validation ✓ 428 warnings, 0 errors. README not regenerated and nothing committed (not requested).
+
+## 2026-08-21
+
+**Prompt:**
+> merge PRs please. Do all the necessary checks first
+
+**Answered:** One open PR: #11 (`kiranjinnil`, "Add Thridy to Icons", self-disclosed author, 1 file / +14 in `data/01-assets/icons/01-icons.yml`). Checks run before merge: closed enums all valid (`license: Free`, `entry_type: asset-source`, `output: [motion-graphics, illustration]`, `platform: web`) and match the §01 asset-source-website convention; `readme_tags` = 2 (cap); chunk size 29/50; catalog-wide dupe grep for `thridy` = 1 hit (the new entry); URL live (200, title confirms "13,000+"); license page verified (free, no signup, no attribution); style clean (no em-dash, no banned words, <300 chars). No CI ran on the PR (0 check-runs, `mergeable_state: unstable` — first-time contributor needs run approval), so CI parity was run locally: `node scripts/validate.js` -> `✓ Validation passed. 428 warnings` (identical to main baseline, 0 new), `render.js` exit 0, `export-data.js` exit 0. Merged with a real `--no-ff` merge commit (d98c067) and pushed; PR #11 auto-closed as merged. Open PRs now 0. README NOT regenerated (asked separately). `gh` CLI still not installed; used the GitHub REST API + plain git.

project.md

@@ -350,6 +350,8 @@ Facet groups are separate **list** properties (`facet_license: ['Free']`), not o
 
 ## 11. Pending tasks / known work
 
+- 2026-08-21: Merged **PR #11** (Thridy, free 3D icon library) into §01 `icons/01` (28→29). Pre-merge checks all green (enums, dupe, URL 200, license page, style); PR CI never ran (first-time-contributor approval gate) so CI parity ran locally: validate ✓ 428, render exit 0, export exit 0. Real `--no-ff` merge commit `d98c067` pushed; PR auto-closed. Open PRs 0. **OPEN:** `README.md` not regenerated for this entry (site + README still lack Thridy). `gh` CLI still not installed; PR work done via GitHub REST API + plain git.
+
 - 2026-08-19: Added **MipMap** (drone mapping photogrammetry) to §10 `photogrammetry-scanning/03` (31→32), beside Pix4D. Platform `[win, mac]` inherited from Pix4D and NOT verified (site is JS-rendered). Full `check-links.js` write run over 3,543 URLs: OK 3,426, broken 58, unreachable 59, redirects 0, **0 new auto-deprecations** — the 58 broken are the same long-standing set. Report `_maintenance/link-check-2026-08-18.md` (script stamps UTC, so a local-2026-08-19 run is dated 08-18). Validation ✓ 428 warnings, 0 errors. **OPEN:** 58 broken URLs still need human triage; `scripts/recheck-unreachable.js` not yet run against the 59 unreachable (mostly Adobe/Autodesk bot-blocks). README not regenerated; nothing committed.
 
 - 2026-08-04: **CI maintenance workflows were failing silently for months.** GitHub issues/PRs open: 0; the real breakage was in Actions. `release-watch.yml` failed every run since May 2026 (`git add _maintenance/release-state.json` on a gitignored path exits 1) — fixed by changing `.gitignore` to `_maintenance/*` + `!_maintenance/release-state.json`; the glob form is required because git cannot re-include a file whose parent dir is excluded. `freshness.yml` succeeded in May with a direct-commit step, then failed Jun/Jul/Aug after the swap to `peter-evans/create-pull-request@v6` — its `add-paths` listed gitignored `_maintenance/**` globs; trimmed to `data/**`. `link-check.yml` used `content-filepath: ..._$(date +%Y-%m-%d).md`, which never expands inside a `with:` value; now computed as a step output (this is why no link-check issue was ever filed despite green steps). **OPEN for user:** freshness may additionally need repo setting Settings → Actions → General → "Allow GitHub Actions to create and approve pull requests" (cannot verify without an authenticated token). Confirm on the next scheduled run (1st of month) or a `workflow_dispatch`.