sync beads: bd-v1qc (qmd writer drops trailing newline in code blocks, issue #173)

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

Author: cscheid

.beads/issues.jsonl

@@ -272,6 +272,7 @@
 {"id":"bd-ulgr","title":"Design JS dependency handling for Quarto 2 HTML output (Bootstrap JS and beyond)","description":"Quarto 2 HTML output currently emits correct DOM for Bootstrap-based features (navbars with dropdowns, collapsible navigation, future tabset panels, callout collapse, crossref popovers, dark-mode toggle, search) but ships no JavaScript dependencies, so interactive features do not function. Decide and implement a strategy for declaring, collecting, and emitting JS dependencies through the render pipeline.\n\nScope includes: vendor vs CDN choice; who declares dependencies (theme? feature transform? template?); how deps are collected into the final HTML; interaction with the resource-collector stage; how WASM / hub-client renders get these deps; version pinning aligned with the bundled Bootstrap SCSS in quarto-sass.\n\nOutline document: claude-notes/plans/2026-04-18-html-js-deps-design.md\n\nBlocks full UX of bd-imiw (navbar dropdowns, collapse) once implemented.","status":"open","priority":1,"issue_type":"feature","created_at":"2026-04-18T19:33:38.359516Z","created_by":"cscheid","updated_at":"2026-04-18T19:33:38.359516Z","source_repo":".","compaction_level":0,"original_size":0}
 {"id":"bd-uy3z","title":"pampa: wire Meta and Pandoc filter callbacks in typewise dispatch","description":"Discovered while testing bd-o8pr Phase 3 (Lua filter resource channel).\n\npampa::lua::filter::filter_names lists 'Pandoc', 'Doc', 'Meta', 'Block', 'Blocks', 'Inline', 'Inlines' as recognized filter handler names — they get copied from globals into the filter table. But the typewise traversal in apply_typewise_filter does not actually invoke them.\n\nThis means a Lua filter that does:\n  function Meta(meta) ... end\nor:\n  function Pandoc(doc) ... end\n\nis silently ignored. Para/Str/Header/etc. all work.\n\nThis blocks:\n- Lua-filter tests for the bd-o8pr 'additivity defense' (filter mutates meta.resources): the unit test in resource_report.rs covers the logic, but an E2E test through q2 render needs Meta callback support.\n- General pandoc filter compatibility: many real filters do all their work in Pandoc(doc) or Meta(meta).\n\nReferences:\n- crates/pampa/src/lua/filter.rs:308 (filter_names list)\n- crates/pampa/src/lua/filter.rs:1730 (apply_typewise_filter)\n- crates/quarto-core/tests/project_resources.rs (filter_removing_meta_resources_does_not_drop_author_declaration test note)","status":"open","priority":3,"issue_type":"feature","created_at":"2026-05-03T19:54:04.819189Z","created_by":"cscheid","updated_at":"2026-05-03T19:54:04.819189Z","source_repo":".","compaction_level":0,"original_size":0,"dependencies":[{"issue_id":"bd-uy3z","depends_on_id":"bd-o8pr","type":"discovered-from","created_at":"2026-05-03T19:54:04.819189Z","created_by":"cscheid","metadata":"{}","thread_id":""}]}
 {"id":"bd-v0zm","title":"Cargo: upgrade reqwest v0.12.28 → v0.13.3","description":"Major upgrade surfaced by cargo-upgrade survey 2026-05-04. Current 0.12.28 is range-pinned in workspace; latest is 0.13.3. Type: pre-1.0 minor (semver-breaking). Review changelog and bump deliberately. See claude-notes/plans/2026-05-04-cargo-upgrade-survey.md and bd-hb8h.","status":"closed","priority":3,"issue_type":"chore","created_at":"2026-05-04T18:15:54.971306Z","created_by":"cscheid","updated_at":"2026-05-04T20:30:45.491686Z","closed_at":"2026-05-04T20:30:45.491526Z","close_reason":"merged: d5d9b5d1","source_repo":".","compaction_level":0,"original_size":0,"labels":["cargo","deps"],"dependencies":[{"issue_id":"bd-v0zm","depends_on_id":"bd-hb8h","type":"discovered-from","created_at":"2026-05-04T18:16:05.108448Z","created_by":"cscheid","metadata":"{}","thread_id":""}]}
+{"id":"bd-v1qc","title":"qmd writer drops trailing newline inside code blocks (issue #173)","description":"The qmd writer in crates/pampa/src/writers/qmd.rs:628-634 emits the closing fence directly after the content text when the text already ends with '\\n', collapsing one content-line boundary. As a result, a CodeBlock whose AST text ends in '\\n' round-trips with the trailing newline removed.\n\nTriage doc: claude-notes/issue-reports/173/triage.md (on branch issue-173).\nRepro: claude-notes/issue-reports/173/repro.sh.\n\nThe pampa reader matches Pandoc exactly on this — verified across seven cases including CommonMark spec example 100. So the AST contract is correct; only the writer is wrong. Fix scope is in the triage doc.\n\nReporter: @rundel. GH issue: https://github.com/quarto-dev/q2/issues/173.","status":"open","priority":1,"issue_type":"bug","created_at":"2026-05-11T20:52:28.394443Z","created_by":"cscheid","updated_at":"2026-05-11T20:52:28.394443Z","source_repo":".","compaction_level":0,"original_size":0}
 {"id":"bd-v30t","title":"Phase 6 — Cross-document link rewriting","description":"Rewrite body-content [link](other.qmd) references in the AST to relative output URLs. Sub-plan: claude-notes/plans/2026-04-24-websites-phase-6.md. Adds LinkRewriteTransform, resolve_doc_relative_href helper, page_url_for resolver method, and RenderContext.resource_resolver field.","status":"closed","priority":1,"issue_type":"feature","created_at":"2026-04-27T12:53:10.861557Z","created_by":"cscheid","updated_at":"2026-04-27T13:42:23.653492Z","closed_at":"2026-04-27T13:42:23.653203Z","close_reason":"Phase 6 complete on feature/websites: LinkRewriteTransform + resolve_doc_relative_href helper + page_url_for on ResourceResolverContext + resource_resolver field on RenderContext/StageContext. 49 new tests, full xtask verify green. Sub-plan: claude-notes/plans/2026-04-24-websites-phase-6.md. Follow-ups: bd-p4sc, bd-fo1r, bd-nb32, bd-j3a0, bd-gdrv, bd-td2a.","source_repo":".","compaction_level":0,"original_size":0,"dependencies":[{"issue_id":"bd-v30t","depends_on_id":"bd-0tr6","type":"parent-child","created_at":"2026-04-27T12:53:10.861557Z","created_by":"cscheid","metadata":"{}","thread_id":""}]}
 {"id":"bd-v8gx","title":"Rename flush_site_libs to flush_project_artifacts","description":"`flush_site_libs` in `crates/quarto-core/src/project/website_post_render.rs` is now called from both `WebsiteProjectType.post_render` (where the name is accurate) and `RenderToHtmlRenderer.render` (where 'site_libs' is misleading — default projects have no site_libs dir). The function body has always been general (it just iterates project_artifacts and calls `resolver.on_disk_path_for(Project, ...)`). Rename + update one error message for clarity. Pure naming hygiene.","status":"open","priority":4,"issue_type":"chore","created_at":"2026-05-01T22:49:10.919652Z","created_by":"cscheid","updated_at":"2026-05-01T22:49:10.919652Z","source_repo":".","compaction_level":0,"original_size":0,"dependencies":[{"issue_id":"bd-v8gx","depends_on_id":"bd-87fu","type":"discovered-from","created_at":"2026-05-01T22:49:10.919652Z","created_by":"cscheid","metadata":"{}","thread_id":""}]}
 {"id":"bd-varx","title":"L9 follow-up: hoist append_to_rendered_header + escape_html_attr to a shared util","description":"L9's link_inject.rs duplicates two small helpers from website_favicon.rs (append_to_rendered_header to write into rendered.includes.header, and escape_html_attr for double-quoted attribute values). Both are now used by ≥2 transforms (favicon + listing-feed-link). Hoist to a shared module under crates/quarto-core/src/transforms/ or website_config.rs and have both call sites use the shared form.","status":"open","priority":4,"issue_type":"task","created_at":"2026-05-08T17:34:04.030827Z","created_by":"cscheid","updated_at":"2026-05-08T17:34:04.030827Z","source_repo":".","compaction_level":0,"original_size":0,"dependencies":[{"issue_id":"bd-varx","depends_on_id":"bd-o90m","type":"discovered-from","created_at":"2026-05-08T17:34:04.030827Z","created_by":"cscheid","metadata":"{}","thread_id":""}]}