Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
A Swift 6+ package can depend on older Swift packages, but not the reverse — so the main package must be on Swift 6 before adopting Swift 6.3-only libraries like swift-openapi-generator and swift-subprocess. Swap Phase 2 and Phase 3 accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…omponent enforcement - Create issue #53: Enforce Component-Based Plot API (restrict direct Node creation to internal, force Component protocol usage throughout) - Link #40 (Ink→swift-markdown), #41/#46 (ShellOut→swift-subprocess in Tagscriber vs Publish/NPMPublishPlugin), #43 (SyndiKit main branch upgrade), #44 (swift-argument-parser→swift-configuration), #45 (Prch removal), #47 (MarkdownGenerator→local subrepo) to their Phase 1/3 deliverables - Add Future Work section for #49 (AT Protocol), #50 (GitHub Pages), #51 (node-swift research) - Update MarkdownGenerator in compatibility matrix: KEEP→BRING LOCAL (#47) - Note #43 SyndiKit upgrade as prerequisite for Phase 2 macOS 13 minimum Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…program path Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Merges the detailed technical content from .claude/PRD-V2-MIGRATION.md into PRD.md — correct phase ordering (Swift 6 before OpenAPI), dependency research table, publishing infra architecture, and new issue #51. Deletes PRD-V2-MIGRATION.md. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
leogdion
commented
Apr 13, 2026
Comment on lines
+67
to
+69
| | #3 | Add Additional Local Storage Options | Open | | ||
| | #4 | Add Main Actor to Swift 6 Article Solution | Open | | ||
| | #13 | Clarify String vs Reference design choice in MistKit article | Open | |
Member
Author
There was a problem hiding this comment.
these issues are not part of the migration but just content edits. mark so in this document and apply a label.
Member
Author
There was a problem hiding this comment.
move these to the bottom
Comment on lines
+90
to
+95
| | #19 | Implement FAQ Schema Markup in `PiHTMLFactory` | P0-critical | In Progress | | ||
| | #20 | Implement HowTo Schema Markup in `PiHTMLFactory` | P1-high | Open | | ||
|
|
||
| **Implementation files:** | ||
| - `Sources/BrightDigitSite/Nodes/PiHTMLFactory.HTML.swift` — head generation | ||
| - `Sources/BrightDigitSite/PiHTMLFactory.swift` — main factory |
Member
Author
There was a problem hiding this comment.
Is PiHTMLFactory the protocol? Verify FAQ and HowTo are appropriate for a company site. If not find appropriate counter parts.
|
|
||
| | # | Title | Status | | ||
| |---|-------|--------| | ||
| | #38 | Swift 6 Language Mode + Component Migration + Mermaid Support | Open | |
Member
Author
There was a problem hiding this comment.
Add an issue to upgrade Tailwind and create a library for easy tailwind access
| | # | Title | Notes | | ||
| |---|-------|-------| | ||
| | #50 | Migrate to GitHub Pages | Currently deployed via Netlify | | ||
| | #49 | Support AT Protocol | Reference: [A Social Filesystem](https://overreacted.io/a-social-filesystem/) | |
Member
Author
There was a problem hiding this comment.
This should be a prerequisite for PublishKit?
| |---|-------|-------| | ||
| | #34 | Remove or repurpose Import/Wordpress XML files | Clean up leftover import artifacts | | ||
| | #1 | Skip Campaign Download For Existing Newsletters | May be superseded by Phase 6 Buttondown migration; keep for now | | ||
| | #51 | Research node-swift | Evaluate [kabiroberai/node-swift](https://github.com/kabiroberai/node-swift); may affect NPMPublishPlugin long-term | |
Member
Author
There was a problem hiding this comment.
this should be prerequisite for nom plugin
| | # | Title | Notes | | ||
| |---|-------|-------| | ||
| | #34 | Remove or repurpose Import/Wordpress XML files | Clean up leftover import artifacts | | ||
| | #1 | Skip Campaign Download For Existing Newsletters | May be superseded by Phase 6 Buttondown migration; keep for now | |
Member
Author
There was a problem hiding this comment.
this should be part of buttondownkit integration
- Move #18 (seomachine) from Phase 0 to Phase 1A (AI integration) - Mark Phase 0B explicitly as non-migration content edits; call out article-edit label - Frame AI-CITE as Swift site-building integration (BrightDigitSite/Publish) - Add schema type verification note for FAQ/HowTo in Phase 1A - Add re-evaluation note to Phase 1B (post-migration) - Mark Phase 1D as post-migration tasks - Update all Swift 6 references to Swift 6.3; remove StrictConcurrency task (not needed in 6.3) - Expand Kanna/MarkdownGenerator rows with usage context and research notes - Note #46 (NPMPublishPlugin) should leverage #51 (node-swift) before implementing - Add optional #1 to Phase 4 success criteria - Add TBD Tailwind upgrade issue to Phase 5 - Add TBD form integration issue to Phase 7 - Note #49 as potential PublishKit prerequisite - Move #1 to ButtondownKit integration (Phase 6) context - Mark #51 as prerequisite for Phase 4 #46 - Update issue count table (Phase 0→2, Phase 1→11, Total→38) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
leogdion
commented
Apr 13, 2026
Comment on lines
+101
to
+105
| | #21 | Optimize Mise Setup Guide for AI-CITE | P1-high | Open | | ||
| | #22 | Optimize Best Backend Article for AI-CITE | P1-high | Open | | ||
| | #26 | Optimize iOS CI/CD Article for AI-CITE | P1-high | Open | | ||
| | #27 | Optimize iOS Architecture Article for AI-CITE | P1-high | Open | | ||
| | #28 | Optimize Remaining Priority Articles (Batch) | P1-high | Open | |
Member
Author
There was a problem hiding this comment.
move these to the bottom
Comment on lines
+67
to
+69
| | #3 | Add Additional Local Storage Options | Open | | ||
| | #4 | Add Main Actor to Swift 6 Article Solution | Open | | ||
| | #13 | Clarify String vs Reference design choice in MistKit article | Open | |
Member
Author
There was a problem hiding this comment.
move these to the bottom
Comment on lines
+119
to
+120
| | #24 | YouTube Video Content Strategy | P2-medium | Open | | ||
| | #25 | Create Unique BrightDigit Frameworks/Methodologies | P2-medium | Open | |
Member
Author
There was a problem hiding this comment.
move them to the correct phase
- Update dependency chain: Phase 1A requires Phase 3; Phase 1C independent - Add schema integration research note to Phase 1A - Move Phase 0B (article edits) to new Post-Migration Content Tasks section - Move Phase 1B (article optimization) to Post-Migration Content Tasks section - Move Phase 1D (#24, #25) to Phase 5 (post-migration AI-CITE content strategy) - Add TBD issue to Phase 3: investigate decoupling content updates from Swift build - Update issue count table to reflect restructuring Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…nts, NPM, and content/code separation Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Use SchemaProvider protocol in BrightDigitSite for Phase 1; promote to PageContent in Phase 5 during component migration. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…a work Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…itSite implements schemaMarkup: String? belongs on PageContent in PublishType (the protocol layer), not as a workaround protocol in BrightDigitSite. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Phase 1A schema work is additive to PublishType and needs no Swift 6.3 upgrade. Phase 1C validation cannot run before Phase 1A schema is shipped. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Avoids doing AI-CITE code changes across a Swift 6.3 boundary. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Monorepo cleanup → Swift 6.3 → AI-CITE → OpenAPI → Subrepos/Components → Publishing → Platform → Cleanup Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…m conflict Package@swift-6.0.swift on main handles Swift 6.3 compatibility automatically. #43 is recommended before Phase 2 for clean concurrency, not a hard blocker. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
SwiftSoup is pure Swift, Linux-compatible, covers all Kanna usage in Tagscriber/KannaMarkdownGenerator.swift (tag names, text, attributes, child selection via CSS selectors instead of XPath). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.
No description provided.