Skip to content

docs(batch): say what merge-tracker does with a malformed summary file - #159

Open
vliggio wants to merge 1 commit into
career-ops-hq:mainfrom
vliggio:claude/career-ops-3706-docs-fd766e
Open

vliggio wants to merge 1 commit into
career-ops-hq:mainfrom
vliggio:claude/career-ops-3706-docs-fd766e

Conversation

@vliggio

@vliggio vliggio commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

Follow-up to #91 (which closed #62). That change fixed the description of the addition files; this one covers the other user-visible half of career-ops-hq/career-ops#3706.

What was missing

Step 6 of the batch guide lists what merge-tracker.mjs does with the files in batch/tracker-additions/ (read, dedupe, add, archive) and says nothing about a file it cannot read. Since career-ops-hq/career-ops#3706, a malformed addition is refused: the script prints a ⚠️ Skipping <file>: … warning that names the problem and adds no tracker row for that offer. A reader whose batch finished with one row fewer than expected had no line in the guide to explain it.

What it says now

One paragraph after the --dry-run block in Step 6:

If a summary file is malformed, the script skips it: it prints a Skipping warning that names the file and the problem, and adds no row for that offer. The preview prints the same warnings, so you can catch one before anything is archived.

Same paragraph in all four copies:

  • content/docs/introduction/guides/batch-evaluate-offers.mdx
  • content/docs/introduction/guides/batch-evaluate-offers.es.mdx
  • content/docs/introduction/guides/batch-evaluate-offers.fr.mdx
  • content/docs/introduction/guides/batch-evaluate-offers.de.mdx

translationHash in ES, FR and DE goes from d7a6ccee0dc73e47 to ee84e9227a63d238 (node .i18n/hash.mjs on the English file).

Checked against the core

Read from merge-tracker.mjs on career-ops main today, not run:

  • The parse refusals all log ⚠️ Skipping ${filename}: … and return no addition; the merge loop counts the file as skipped and moves on.
  • --dry-run runs the same parse loop, so it prints the same warnings. Only the tracker write and the move to merged/ are gated on it.

Validation

  • npm run types:check: passes.
  • npm run build: passes.
  • scripts/verify-agent-layer.mjs against the local build: 1 failure, /changelog.md release headings lost their subject. That route renders from the release list src/lib/releases fetches, and this change touches only four .mdx files, so I believe it is the local environment rather than this diff. CI will say.
  • The .md twin of the guide carries the new paragraph in all four languages on the local build.

AI assistance

Claude Code (Claude Opus 5.5), in a session started by @vliggio. The ES, FR and DE paragraphs were translated by the agent, not by the .i18n/translate.mjs pipeline.

Human review

  • Diff read
  • Page rendered
  • Translations checked
  • Guard green

🤖 Generated with Claude Code

Step 6 listed what merge-tracker.mjs does with the files in
batch/tracker-additions/ but not what happens when one cannot be read.
Since career-ops-hq/career-ops#3706 a malformed addition is skipped with
a Skipping warning that names the file and the problem, and no tracker
row is added. Say so, and point at --dry-run, which prints the same
warnings before anything is archived. EN, ES, FR and DE; translationHash
restamped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@vliggio
vliggio requested a review from santifer as a code owner October 10, 2026 01:10
@vercel

vercel Bot commented Oct 10, 2026

Copy link
Copy Markdown

@vliggio is attempting to deploy a commit to the Career Ops OSS Program Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai

coderabbitai Bot commented Oct 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 33 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository: career-ops-hq/career-ops-docs/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 1ba3e939-3cdc-4ba6-b809-c918604a1efc

📥 Commits

Reviewing files that changed from the base of the PR and between f217ca5 and e75c6bf.


📒 Files selected for processing (4)
  • content/docs/introduction/guides/batch-evaluate-offers.de.mdx
  • content/docs/introduction/guides/batch-evaluate-offers.es.mdx
  • content/docs/introduction/guides/batch-evaluate-offers.fr.mdx
  • content/docs/introduction/guides/batch-evaluate-offers.mdx


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

This branch has not been deployed

No deployments
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.

Batch guide: "One-line tracker entries" goes stale when the headed addition TSV lands (career-ops#3706)

1 participant