Skip to content

[site] Review pass on the two vibe pages, and a tidy header nav - #12

Merged
ctr26 merged 3 commits into
masterfrom
claude/anthropic-life-science-roles-5psay3
Oct 4, 2026
Merged

ctr26 merged 3 commits into
masterfrom
claude/anthropic-life-science-roles-5psay3

Conversation

@ctr26

@ctr26 ctr26 commented Oct 4, 2026 •

Copy link
Copy Markdown
Owner

Summary

Follow-up to merging #1 and #2. An adversarial review of the two pages against the CV, the house style, and Minima's behaviour found two high items and a long tail of style; this PR applies it.

Nav (_config.yml, index.md, assets/main.scss, cv-print.css)

  • header_pages: [vibe.md, vibe-coding.md]. Minima lists every titled page in the header, so after feat: add vibe coding page #1 and Add vibe coding for scientists guide #2 the nav read as the CV's 70-character job title plus two near-duplicate "Vibe Coding" links
  • The nav was also where the job title showed on screen, and what the print CSS used as the subtitle. It now lives in index.md as a raw <p class="jobtitle"> (kramdown and pandoc both keep it); the print CSS hides the nav and styles that paragraph as the subtitle; cv-print.css hides it because pandoc already prints the title as the document subtitle

vibe.md → "How I Work"

  • Retitled to what it describes (a working-style guide, not a vibe-coding guide), so the nav no longer shows two "Vibe Coding" entries
  • [Back to CV](./) removed; with permalink: /vibe/ it linked to itself
  • "Last updated: March 2026" removed
  • Agent-instruction items ("Each tool call, file edit, agent spawn is self-contained", "Thinker/Worker separation") rewritten or dropped; the page reads as a person's guide rather than a reformatted CLAUDE.md
  • Cross-link to the scientists' guide, and one paragraph reconciling "disposable code" with "no LLM-looking code"

vibe-coding.md

  • Bio corrected to Senior Machine Learning Scientist at Valence Labs, Recursion, generative and agentic models for biology
  • Native installer shown first (brew and npm noted), docs URL updated to code.claude.com
  • One line acknowledging Karpathy's sense of "vibe coding" and why the page means something more careful
  • "force multiplier", "transformative", "dramatically" cut; productivity claim softened to "in my experience"
  • Cross-link to How I Work; layout: default with a manual H1 to match the CV; description added

House style

  • No colons or em dashes in prose on either page (code fences excepted); checked by script

Testing

  • CI artifact for the first commit: nav shows only "How I Work" and "Vibe Coding for Scientists", one H1 per page, descriptions and cross-links present, #fundamentals anchor resolves
  • Browser print of the built page with the new CSS: 2 A4 pages, header reads name, job title, contacts
  • make pdf: 2 pages, same header

🤖 Generated with Claude Code

https://claude.ai/code/session_01JFVYogXC9MgS2EuFBvrZog

vibe.md becomes "How I Work", which is what it describes, drops the
self-linking "Back to CV" and the stale "Last updated", and loses the
agent-instruction items. vibe-coding.md gets the current title and
employer in the bio, the native installer, the current docs URL, a
line acknowledging Karpathy's sense of the term, and softer claims.
Both pages cross-link, reconcile "disposable code" with "no
LLM-looking code", carry a description, use the default layout, and
follow the house style (no colons or em dashes in prose).

_config.yml sets header_pages to the two pages, so the CV's long job
title no longer appears as a nav link.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JFVYogXC9MgS2EuFBvrZog
@ctr26
ctr26 marked this pull request as ready for review October 4, 2026 07:02
Copilot AI balanced review requested due to automatic review settings October 4, 2026 07:02
With header_pages set, the Minima nav stopped showing index.md's
title, which the screen relied on for the job title and the print CSS
used as the subtitle. A raw <p class="jobtitle"> at the top of
index.md carries it in both kramdown and pandoc; the print CSS hides
the nav outright and styles the paragraph as the subtitle, and
cv-print.css hides it because pandoc already prints the title as the
document subtitle. Both print routes stay at 2 A4 pages.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JFVYogXC9MgS2EuFBvrZog

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

Two technical claims remain inaccurate, and one sentence requires a grammatical correction.

Review effort: Balanced
Findings: 3 Low severity

Open (3)
What changed in this PR

Refines two guidance pages and simplifies the Minima header navigation.

Changes:

  • Retitles and streamlines the working-style guide.
  • Updates the scientist-focused Claude Code guide and biography.
  • Restricts header navigation to the two guidance pages.
File Description
_config.yml Defines explicit header navigation pages.
vibe.md Reframes the page as “How I Work.”
vibe-coding.md Revises setup, guidance, cross-links, and biography.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread vibe-coding.md Outdated
Comment thread vibe.md Outdated
Comment thread vibe.md
Worktrees isolate working trees, not merges, and Claude Code reads
files as it needs them rather than holding the whole repository.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JFVYogXC9MgS2EuFBvrZog
@ctr26
ctr26 merged commit 7683db5 into master Oct 4, 2026
2 checks passed
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.

3 participants