Adopt federated Markdown rendering from template 0.12.1 #64

Merged

erik merged 1 commit from fix/task-0f901647-federated-markdown into main

2026-10-29 21:05:08 -05:00

erik commented

2026-10-29 21:02:54 -05:00

Owner

Summary

Adopt the released template 0.12.1 safe Markdown renderer for notes and link commentary; retain article permalink-only content.
Share rendering across Create/Update/outbox and negotiated individual ActivityPub object requests while preserving custom actor, inbox, authentication and exact-version API routes.
Adapt release regressions to this site, including formatting, unsafe URLs/raw HTML, canonical links, linked images, source/API preservation, draft privacy, JSON-LD/HEAD and actor-less fallback.
Document adoption and separately authorized rollout; no migrations, dependency changes, deployment or historical note edits/backfill.

Verification and delivery

./scripts/pre-pr.sh: formatting, typecheck, build, generated CSS unchanged, 712 tests passing across 69 files.
npm run test:ci: 572 tests passing across 56 files; integration coverage remains local-only.
git diff --check: clean.
Independent read-only review at 1de4c57 found no actionable issues; transcript summarized in the PR comment.
Exact-head PR CI passed: https://forge.caradoc.com/erik/slugkit.com/actions/runs/76/jobs/0 .
Merged as 8ad08a1; local main synchronized and task branch removed locally/remotely.

Task: task-0f901647.
Reference: template 0.12.1 at 9781aff0353e3c4fb91f06fdc9550386a25d1f97 / upstream PR #313.

Production deployment and existing-note recovery remain separately authorized and were not performed.

## Summary - Adopt the released template 0.12.1 safe Markdown renderer for notes and link commentary; retain article permalink-only content. - Share rendering across Create/Update/outbox and negotiated individual ActivityPub object requests while preserving custom actor, inbox, authentication and exact-version API routes. - Adapt release regressions to this site, including formatting, unsafe URLs/raw HTML, canonical links, linked images, source/API preservation, draft privacy, JSON-LD/HEAD and actor-less fallback. - Document adoption and separately authorized rollout; no migrations, dependency changes, deployment or historical note edits/backfill. ## Verification and delivery - `./scripts/pre-pr.sh`: formatting, typecheck, build, generated CSS unchanged, 712 tests passing across 69 files. - `npm run test:ci`: 572 tests passing across 56 files; integration coverage remains local-only. - `git diff --check`: clean. - Independent read-only review at 1de4c57 found no actionable issues; transcript summarized in the PR comment. - Exact-head PR CI passed: https://forge.caradoc.com/erik/slugkit.com/actions/runs/76/jobs/0 . - Merged as 8ad08a1; local main synchronized and task branch removed locally/remotely. Task: task-0f901647. Reference: template 0.12.1 at 9781aff0353e3c4fb91f06fdc9550386a25d1f97 / upstream PR #313. Production deployment and existing-note recovery remain separately authorized and were not performed.

erik added 1 commit

2026-10-29 21:02:54 -05:00

fix: render federated Markdown as safe HTML

CI / verify (pull_request) Successful in 47s

Details

1de4c57f84

erik commented

2026-10-29 21:05:02 -05:00

Author

Owner

Independent read-only review of commit 1de4c57 (separate pi reviewer, following pr-review skill):

No actionable issues found in PR #64 at 1de4c57.
Reviewed the complete supplied diff and relevant renderer, federation, routing, authentication and privacy code.
Verification limits: read-only review; tests were not rerun. Supplied logs show 712 full-suite tests and 572 CI-suite tests passing. Remote CI and production behavior remain unverified.

Maintainer-agent verification: remote CI / verify (pull_request) passed for exact head 1de4c57f84 (47s): https://forge.caradoc.com/erik/slugkit.com/actions/runs/76/jobs/0 . Full local ./scripts/pre-pr.sh and deterministic test:ci passed. No production checks, releases, deployment or existing-note edits/backfill performed.

Review transcript: /tmp/task-0f901647-review.log (local). Delivery gates satisfied; proceeding to merge and branch cleanup.

Independent read-only review of commit 1de4c57 (separate pi reviewer, following pr-review skill): > No actionable issues found in PR #64 at 1de4c57. > Reviewed the complete supplied diff and relevant renderer, federation, routing, authentication and privacy code. > Verification limits: read-only review; tests were not rerun. Supplied logs show 712 full-suite tests and 572 CI-suite tests passing. Remote CI and production behavior remain unverified. Maintainer-agent verification: remote CI / verify (pull_request) passed for exact head 1de4c57f84309badb3ef2f32755f1aa33bef8a4f (47s): https://forge.caradoc.com/erik/slugkit.com/actions/runs/76/jobs/0 . Full local ./scripts/pre-pr.sh and deterministic test:ci passed. No production checks, releases, deployment or existing-note edits/backfill performed. Review transcript: /tmp/task-0f901647-review.log (local). Delivery gates satisfied; proceeding to merge and branch cleanup.

erik merged commit 8ad08a10ab into main

2026-10-29 21:05:08 -05:00

erik deleted branch fix/task-0f901647-federated-markdown

2026-10-29 21:05:08 -05:00

erik referenced this pull request from a commit

2026-10-29 21:05:09 -05:00

Merge pull request 'Adopt federated Markdown rendering from template 0.12.1' (#64) from fix/task-0f901647-federated-markdown into main