Skip to content

Claude/fix acronym search b la lh#18

Merged
klappy merged 2 commits intomainfrom
claude/fix-acronym-search-BLaLH
Feb 6, 2026
Merged

Claude/fix acronym search b la lh#18
klappy merged 2 commits intomainfrom
claude/fix-acronym-search-BLaLH

Conversation

@klappy
Copy link
Copy Markdown
Owner

@klappy klappy commented Feb 6, 2026

Note

Low Risk
Primarily documentation and generated-content updates; main risk is broken links/navigation or stale generated artifacts rather than runtime behavior changes.

Overview
Refreshes the generated documentation export/index and expands navigability and terminology clarity across the repo.

Adds a new docs/CONTENT-MAP.md comprehensive index (including apocrypha) and wires it into the root README and docs/README.md discovery section. Promotes Epistemic Surface Extraction (ESE) into Canon via new canon/epistemic-surface-extraction.md, marks the apocrypha version as PROMOTED, and updates cross-links.

Introduces new Canon content: multiple Tier-1 constraints, a new definition (CST), and a new canon/methods/ section with several method docs; also updates ODD/oddkit entry points (frontmatter/title/tags and callouts) to explicitly expand “ODD = Outcomes-Driven Development” and improve searchability. Updates the odd-teaser lane documentation (new HISTORY.md, KICKOFF.md, behavior.md, and PRD bump to v1.2 requiring real Claude API + worker proxy + rate limiting/streaming) and bumps package version/lockfile plus regenerated public/ compiled outputs (docs.json, manifest.json, public/content/*, and book export).

Written by Cursor Bugbot for commit ee9c7ad. This will update automatically on new commits. Configure here.

The .husky/pre-commit hook was tracked as 100644 (not executable).
Combined with agent environments skipping npm install (so husky's
prepare script never set core.hooksPath), the hook was silently
dead — which is why PRs #15 and #16 shipped without updated
docs.json, manifest.json, or content sync.

This commit sets the file to 100755. The hook content is unchanged.

https://claude.ai/code/session_01Ht8m1Nd7dMgaHNf4qswqgY
Generated by npm install which was needed to activate husky
and verify the pre-commit hook works.

https://claude.ai/code/session_01Ht8m1Nd7dMgaHNf4qswqgY
@klappy klappy merged commit 637dbeb into main Feb 6, 2026
3 checks passed
@klappy klappy deleted the claude/fix-acronym-search-BLaLH branch February 6, 2026 04:37
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

Comment thread klappy-dev-book-export.md
## Reference Implementation Requirement (v1.2)

Klappy.dev is the **reference implementation of Observation-Driven Development**.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Wrong acronym expansion: "Observation" instead of "Outcomes"

High Severity

The PRD v1.2 section says Observation-Driven Development but ODD stands for Outcomes-Driven Development everywhere else in the repository. This is particularly ironic since the entire PR is specifically about making the ODD acronym expansion visible and preventing hallucination of the wrong meaning. The incorrect expansion appears in the Reference Implementation Requirement section of the active PRD.

Fix in Cursor Fix in Web

Comment thread klappy-dev-book-export.md
| Field | Value |
|-------|-------|
| **Status** | Active |
| **PRD Version** | v1.1 |
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

README shows PRD v1.1 but PRD is v1.2

Medium Severity

The odd-teaser/README.md Lane Overview table lists PRD Version as v1.1, but the PRD itself was updated to v1.2 in this same commit. This creates a misleading inconsistency for anyone orienting from the README, which is the primary entry point for the lane.

Fix in Cursor Fix in Web

klappy added a commit that referenced this pull request Apr 24, 2026
Graduates the paired pattern observed across aquifer-mcp J-002 -> H11b
(PRs klappy/aquifer-mcp#17 through #20) as tier-2 canon.

Three deciding-argument recurrences under the manual-enforcement reading:
1. PR #18 BootstrapEntityResult + 9 Bugbot findings (3 silent-truth-loss bugs).
2. PR #20 FanOutEntityResult + High-severity ctx ReferenceError.
3. esbuild-transpile-only boundary condition — orchestrator decision to
   treat adversarial review as sole defense when pipeline bypasses types.

Release-validation-gate: Bugbot clean (both HEADs); independent Sonnet 4.6
validator dispatched; all findings dispositioned in closeout comment
#136 (comment)
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