Merge branch 'main' into test/smoke-tests-built-version

Author: cderv

.claude/rules/copyright.md

@@ -0,0 +1,12 @@
+---
+paths:
+  - "**/*.ts"
+  - "**/*.js"
+  - "**/*.lua"
+---
+
+# Copyright headers
+
+New source files get a single current-year header — `Copyright (C) <year> Posit Software, PBC` (e.g. `2026`), not a back-dated `2020-<year>` range. Leave existing files' headers alone unless you're already substantially editing them.
+
+Don't guess the year — read it from the system (`date +%Y`, or the current date stated at session start). Copying a header from an existing file carries a stale year.

.claude/rules/filters/lua-development.md

@@ -42,6 +42,20 @@ local md = require("modules/md")
 import("./modules/patterns.lua")
 ```
 
+### In `import()`ed Files, Prefer the `_quarto.modules` Registry Over a Fresh `require()`
+
+Files loaded via `import()` are textually concatenated into one shared Lua chunk for
+the packaged distribution build, so every top-level `local` they declare — including
+`local x = require("modules/y")` — competes for Lua's hard 200-local-per-function
+limit across *all* inlined files combined. Most `modules/*` are already registered
+in `_quarto.modules` by `modules/import_all.lua` (a single global table assignment,
+so it costs zero additional locals). In an `import()`ed file, reference
+`_quarto.modules.<name>.<fn>(...)` directly instead of adding a new top-level
+`local <name> = require("modules/<name>")`. This doesn't apply inside files loaded
+via `require()` themselves — those get their own independent chunk and aren't
+affected. See [llm-docs/lua-filter-bundle-local-limit.md](../../../llm-docs/lua-filter-bundle-local-limit.md)
+for the full mechanism and why this keeps recurring.
+
 ## Custom Node Patterns
 
 ### Walking Custom Nodes

.claude/rules/testing/test-anti-patterns.md

@@ -31,3 +31,9 @@ A direct `import { quarto } from "src/quarto.ts"` bypasses binary mode (`QUARTO_
 Never create `Project.toml`, `.venv/`, or `renv.lock` in test fixture directories.
 
 **Details:** `llm-docs/testing-patterns.md` → "Shared Test Environments"
+
+## Don't: `Deno.chdir()` inside the test body
+
+`Deno.chdir()` mutates process-global cwd, so a test that changes it can leak into other tests in the same process. The harness already changes and restores the working directory: return the directory from `TestContext.cwd`, create fixtures in `setup`, clean up in `teardown` (examples: `tests/unit/dotenv-config.test.ts`, `tests/smoke/use/template.test.ts`). For a temp directory you don't need to run *from*, use `withTempDir` (`tests/utils.ts`). A test that only needs a *relative* input can pass a path relative to the current cwd without changing it.
+
+**Details:** `llm-docs/testing-patterns.md` → "Working-Directory-Sensitive Tests"

.claude/rules/testing/unit-test-with-r.md

@@ -0,0 +1,15 @@
+---
+paths:
+  - "tests/unit/preview-subdir-knitr-cwd.test.ts"
+---
+
+# R unit test that changes working directory
+
+This test runs a knitr subprocess with cwd set to a scratch directory
+(required to reproduce the bug). R's `.Rprofile` lookup is cwd-exact, so it
+won't pick up `tests/renv` activation there — on CI the R subprocess fails
+with `there is no package called 'rmarkdown'`. `setup()` writes a
+`.Rprofile` into the fixture dir to re-activate renv against the real
+`tests/` project.
+
+**Details:** `llm-docs/testing-patterns.md` → "R Tests That Change Working Directory"

.github/workflows/update-test-timing.yml

@@ -31,11 +31,18 @@ jobs:
           name: timed test file
           path: tests/
 
+      - name: Generate GitHub App token
+        id: app-token
+        uses: actions/create-github-app-token@v2
+        with:
+          app-id: ${{ secrets.QUARTO_DEV_BOT_APP_ID }}
+          private-key: ${{ secrets.QUARTO_DEV_BOT_APP_PRIVATE_KEY }}
+
       - name: Create Pull Request for new tests timing
         id: cpr
         uses: peter-evans/create-pull-request@v8
         with:
-          token: ${{ secrets.GITHUB_TOKEN }}
+          token: ${{ steps.app-token.outputs.token }}
           add-paths: tests/timing-for-ci.txt
           commit-message: |
             timing for new tests [auto PR]

configuration

@@ -13,10 +13,10 @@
 export DENO=v2.7.14
 # TODO figure out where 0.1.41 apple silicon libs are available
 export DENO_DOM=v0.1.41-alpha-artifacts
-export PANDOC=3.8.3
-export DARTSASS=1.87.0
-export ESBUILD=0.25.10
-export TYPST=0.14.2
+export PANDOC=3.10
+export DARTSASS=1.101.0
+export ESBUILD=0.28.1
+export TYPST=0.15.1
 export TYPST_GATHER=0.2.3
 export VERAPDF=1.28.2
 

dev-docs/feature-format-matrix/qmd-files/css-properties/color/color-fuchsia.qmd

@@ -20,7 +20,7 @@ _quarto:
       ensureTypstFileRegexMatches:
         -
           # note css fuchsia #f012be is not the same as css fuchsia #ff00ff
-          - '\[#set text\(fill: fuchsia\); B\]'
+          - '\[#set text\(fill: fuchsia\);\s*B\]'
         - []
 ---
 

dev-docs/feature-format-matrix/qmd-files/css-properties/color/color-magenta.qmd

@@ -19,7 +19,7 @@ _quarto:
     typst:
       ensureTypstFileRegexMatches:
         -
-          - '\[#set text\(fill: rgb\(255, 0, 255\)\); B\]'
+          - '\[#set text\(fill: rgb\(255, 0, 255\)\);\s*B\]'
         - []
 ---
 

dev-docs/feature-format-matrix/qmd-files/css-properties/color/color-mediumseagreen.qmd

@@ -20,7 +20,7 @@ _quarto:
       ensureTypstFileRegexMatches:
 
         -
-          - '\[#set text\(fill: rgb\(60, 179, 113\)\); B\]'
+          - '\[#set text\(fill: rgb\(60, 179, 113\)\);\s*B\]'
         - []
 ---
 

dev-docs/feature-format-matrix/qmd-files/css-properties/color/color-rgb-hex.qmd

@@ -19,7 +19,7 @@ _quarto:
     typst:
       ensureTypstFileRegexMatches:
         -
-          - '\[#set text\(fill: rgb\("#000066"\)\); B\]'
+          - '\[#set text\(fill: rgb\("#000066"\)\);\s*B\]'
         - []
 ---