docs: add Phase-2 frontend cite/export planning docs#34
Merged
Conversation
Recovered from stale branch claude/clever-colden-617e8b (tip 054df8f, 78 commits behind main). These Phase-2 (frontend cite/export) planning docs were the only unique content on that branch; all its code/test work is already in main. Part 1 of 2. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Final part of the Phase-2 planning salvage from claude/clever-colden-617e8b: the view.js hydration plan, the phase validation strategy, and two GSD todos (one done, one pending). Part 2 of 2. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
The intended \u003c escape had collapsed to a literal < in several places of 02-02-PLAN.md, including the code example .replace(/</g, '<') — a no-op that would have defeated the escape if copied into the implementation. Restore \u003c consistently, matching 02-RESEARCH.md and src/lib/jsonld.js.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #34 +/- ##
=======================================
Coverage 81.47% 81.47%
=======================================
Files 40 40
Lines 2613 2613
Branches 464 464
=======================================
Hits 2129 2129
Misses 126 126
Partials 358 358 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
dknauss
added a commit
that referenced
this pull request
Jun 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Salvages the Phase-2 (frontend cite & export affordances) planning documents that were stranded on the stale local branch
claude/clever-colden-617e8b(78 commits behindmain). All code/test work from that branch was already inmain; these planning docs were its only unique content.Contents
02-RESEARCH.md— research on the viewScript approach, export-utility reuse, progressive enhancement, and XSS escaping fordata-csl02-01-PLAN.md— per-entry export helper (export-single.js) TDD plan02-02-PLAN.md—data-cslsave-markup plan +block.json/webpack wiring02-03-PLAN.md—view.jsfrontend hydration plan02-VALIDATION.md— phase validation strategyNotable fix
The final commit corrects a copy-paste corruption in
02-02-PLAN.md: the intended<XSS-escape had collapsed to a literal<, including the example.replace(/</g, '<')— a no-op that would have silently defeated the escape if copied into implementation. Restored to<, consistent with02-RESEARCH.mdandsrc/lib/jsonld.js.Docs only — no runtime/code changes.
🤖 Generated with Claude Code