Skip to content

Turn on code folding site-wide, as in rme - #88

Merged
dem-extra1 merged 1 commit into
mainfrom
claude/code-fold
Sep 29, 2026
Merged

dem-extra1 merged 1 commit into
mainfrom
claude/code-fold

Conversation

@dem-extra1

Copy link
Copy Markdown
Contributor

Why

rme folds code chunks by default, with a page-level code-tools menu to show or hide all code, so readers see the output first and open the code when they want it. This site had code-tools: true but code-fold: false, so every chunk rendered expanded.

What

In _quarto-website.yml, format: html::

  • code-fold: false -> code-fold: true
  • add code-summary: "Show R code"

code-tools and code-link were already on. The revealjs formats in _quarto-website.yml and _quarto-revealjs.yml already match rme (code-fold: true, code-link: true, code-summary: "[R code]"), so they are unchanged. The handout profile only defines pdf.

🤖 Generated with Claude Code

https://claude.ai/code/session_014YzWJF4E24Edb6RKXbkWaH


Generated by Claude Code

rme folds code chunks by default in its html output, with a page-level
code-tools menu to show or hide all code at once, so readers see the
output first and open the code when they want it. This site had
code-tools on but code-fold off, so every chunk was shown expanded and
the menu's show/hide toggle had nothing to act on.

Set code-fold: true and add rme's code-summary ("Show R code") to the
html format. The revealjs format already matches rme.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014YzWJF4E24Edb6RKXbkWaH
@github-actions

Copy link
Copy Markdown

Caution

Claude review did not run: the configured API credential is unusable.

No review ran at all, so this PR has not been reviewed. This comment exists so that fact is visible from the PR itself rather than only from the run log (gha#543).

The pre-flight check found that every configured API credential secret carries whitespace inside its value, so it cannot be sent as an HTTP Authorization header. The request would be rejected at the door, so the review was not attempted and no money was spent.

This is a repository configuration defect rather than anything about the diff. Nothing about this PR needs to change, and re-running the workflow will fail identically until the secret is repaired.

A repository admin fixes it under Settings -> Secrets and variables -> Actions by re-setting CLAUDE_CODE_OAUTH_TOKEN (or ANTHROPIC_API_KEY) to a single-line value with no spaces or line breaks. The usual cause is pasting a multi-line block -- a PEM key, a JSON credential file, a wrapped terminal copy -- where the token belongs. claude setup-token prints a fresh token to copy.

The Pre-flight credential shape check step in the linked run names which secret is affected and where the whitespace falls, without printing the value itself.

Until an external verdict lands, this PR is not clean. The red require-review is correct rather than a flake, and a self-review stands in for the missing verdict without replacing it.

View run

Copy link
Copy Markdown
Contributor Author

review / claude-review (and so require-review) is red because the repository's Claude credential secret contains whitespace; the pre-flight check stopped before any review ran. This is the same repo-configuration defect that blocked win#82 and win#83, not anything in this two-line _quarto-website.yml change, and re-running will fail identically. A repo admin needs to re-set CLAUDE_CODE_OAUTH_TOKEN (or ANTHROPIC_API_KEY) as a single-line value under Settings -> Secrets and variables -> Actions. Once that's fixed, a re-run should produce the review.

Self-check of the diff: code-fold: false -> true and code-summary: "Show R code" added under format: html:, matching rme; the file parses as valid YAML, and the revealjs blocks already matched rme.


Generated by Claude Code

@dem-extra1
dem-extra1 merged commit 40e0e55 into main Sep 29, 2026
9 of 12 checks passed
@github-actions

Copy link
Copy Markdown
PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-09-29 06:20 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants