Skip to content

Combinatorics bundle + fix for daily-run duplication (per-subtopic numbering & coordination protocol)#18

Draft
DMDaudio wants to merge 2 commits into
mainfrom
claude/nice-feynman-0rsfxh
Draft

Combinatorics bundle + fix for daily-run duplication (per-subtopic numbering & coordination protocol)#18
DMDaudio wants to merge 2 commits into
mainfrom
claude/nice-feynman-0rsfxh

Conversation

@DMDaudio

@DMDaudio DMDaudio commented Jul 23, 2026

Copy link
Copy Markdown
Owner

What does this PR add or change?

Two things: a new content bundle, and a fix for the process problem that has been generating
overlapping, colliding daily-run PRs.

1. New content — Quant counting-probability / combinatorics

First real coverage of combinatorics (only a single probability question existed before):

  • 1 lesson (quant-counting-probability-combinatorics) — fundamental counting principle,
    factorials, permutations vs. combinations (C(n,k) = P(n,k)/k!), identical-item arrangements,
    and together/apart restrictions.
  • 7 problem-solving questions (00010007) with an easy→hard progression: counting principle
    5!C(8,3)LEVEL anagrams → glue method → "at least one" via complement → "no two
    together" via gap method.
  • Distractors keyed to specific errors; worked explanation + two hints each; every answer verified.

2. Fix — stop daily runs duplicating each other

Root cause: daily-run agents each branch off main and never merge, so a run only sees main
and is blind to other in-flight runs. Combined with a section-wide NNNN counter, every run
grabbed the same "next" number and re-seeded the same subtopics — producing ~15 overlapping open
PRs that collide on both subtopic and numbering.

  • Per-subtopic numbering. NNNN is now unique within the subtopic (new subtopics start at
    0001), not section-wide. Two branches now only collide if they target the same subtopic
    which the protocol forbids. Documented in CONTENT_SCHEMA.md and CONTRIBUTING.md. No existing
    files need renaming (they're already unique within their subtopic). This branch's combinatorics
    questions were renumbered 0017002300010007 to follow the new scheme.
  • Coordination protocol (new section in CONTRIBUTING.md): before claiming a subtopic, check
    the repo's open pull requests (not just main); one subtopic per run; prefer genuine gaps;
    keep each run on its own PR.
  • scripts/coverage.mjs — dependency-free report of per-subtopic question counts (by
    difficulty), lesson presence, and gaps from the current checkout; also flags content tagged
    outside the taxonomy. It states up front that it only sees the checkout, so runs must still check
    open PRs.

Limitation: this fixes the go-forward process. The existing open-PR backlog still needs the
maintainer to merge or close it to clear the overlaps already created.

  • Rebuilt docs/data/content.json via node scripts/build.mjs (39 questions, 9 lessons; zero
    warnings
    ).

Type of change

  • New question(s)
  • New lesson(s)
  • Fix to existing content (answer / explanation / tag)
  • Tooling / docs / schema (numbering convention, contributor protocol, coverage script)

Author checklist

  • This is my own original work — not copied or paraphrased from any real GMAT question or prep material.
  • Front-matter follows CONTENT_SCHEMA.md (id matches the filename; topic/subtopic exist in curriculum.md).
  • I solved every question myself and confirmed exactly one answer choice is correct.
  • Each explanation would help someone who got the question wrong, and names the trap where relevant.
  • status is in-review. I did not set my own content to verified.

For reviewers

  • I solved the question(s) without reading the keyed answer first and got the same answer.
  • Exactly one choice is correct; distractors are plausible.
  • Explanation and tags (section / topic / difficulty) are accurate.
  • The per-subtopic numbering change and coordination protocol read sensibly for future runs.
  • If approving: bumped status to verified and added my handle to reviewers.

claude added 2 commits July 23, 2026 09:13
Covers the previously-empty counting-probability/combinatorics subtopic
(only a single probability question existed before):
- Lesson on the fundamental counting principle, factorials, permutations
  vs. combinations, identical-item arrangements, and together/apart
  restrictions.
- 7 original questions (0017-0023) with an easy->hard progression, each
  with a worked explanation, two hints, and distractors keyed to specific
  common errors (add-vs-multiply, permutation-vs-combination, forgetting
  to divide out repeats, at-least-one over-count, gap-method slips).

All status: in-review. Rebuilt docs/data/content.json (zero warnings).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WknFxpevhwGoMJPnoUW8bd
…ocol

Root cause: daily-run agents each branch off main and never merge, so a
run only sees main and is blind to other in-flight runs. Combined with a
section-wide NNNN counter, every run grabbed the same next number and
re-seeded the same subtopics -> ~15 overlapping, colliding open PRs.

Fixes:
- Numbering is now per-subtopic (NNNN unique within the subtopic, new
  subtopics start at 0001) instead of section-wide. Two branches now only
  collide if they target the SAME subtopic, which the protocol forbids.
  Documented in CONTENT_SCHEMA.md and CONTRIBUTING.md. No existing files
  need renaming (they are already unique within their subtopic).
- New 'Automated & daily-run contributions' protocol in CONTRIBUTING.md:
  check OPEN pull requests (not just main) before claiming a subtopic,
  one subtopic per run, prefer genuine gaps, keep each run on its own PR.
- New scripts/coverage.mjs: prints per-subtopic question counts, lesson
  presence, and gaps from the checkout (and flags mistagged content), so a
  run can find gaps fast. It notes it only sees the current checkout.
- Renumbered this branch's combinatorics questions 0017-0023 -> 0001-0007
  to follow the new scheme; rebuilt content.json (zero warnings).

Note: this fixes the go-forward process. The existing open-PR backlog
still needs the maintainer to merge or close it to fully clear the
overlaps already created.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WknFxpevhwGoMJPnoUW8bd
@DMDaudio DMDaudio changed the title Add Quant combinatorics lesson + 7 questions (easy→hard) Combinatorics bundle + fix for daily-run duplication (per-subtopic numbering & coordination protocol) Jul 23, 2026
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