docs(specs): cgis init-ontology design — auto-proposed patterns.yaml (#174)#198
Conversation
…174) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
/gemini review |
|
/guardian review |
There was a problem hiding this comment.
Code Review
This pull request introduces a design specification for cgis init-ontology to auto-propose a patterns.yaml file. The feedback suggests explicitly validating that the SQLite database file exists before opening a connection in both the propose_ontology flow and the MCP tool to prevent SQLite from silently creating empty database files.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
…e (gemini review) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
There was a problem hiding this comment.
Code Review
This pull request introduces a design specification for the cgis init-ontology feature, which automatically proposes a patterns.yaml configuration to simplify onboarding. The feedback suggests updating the proposed flow to explicitly validate the existence of the SQLite database file before opening the store, preventing SQLite from silently creating an empty database file.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
|



Summary
Design spec for #174 (roadmap #179 P0, the adoption-blocker half of the pair with merged #178):
cgis init-ontologyproposes a ready-to-editpatterns.yamlfrom the measured graph — measure-then-label instead of a blank page.Key decisions:
score()against each of the 5 bundled templates via a synthetic DomainConfig — zero new fitting math; layer weights / empty-layer exclusion / unresolved discount inherited from_score_v2, so proposed numbers stay comparable to later drift runs# no template fits (best: X at 0.55)— no forced labels (the httpx mislabel trap from 🗺️ Roadmap: priorities from multi-repo dogfooding (#170–#178) #179); belowmin_nodes/ zero edges → hygiene-only with reasons. Hygiene-only entries also getmeasured+margintolerance so the round-trip guarantee covers cycle-bearing unlabeled domainsanalyze_drifton the same graph → no empty, nothing above its proposed tolerance, gate green by construction; on fixtures and the cgis self-graphinit-ontology(overwrite-refusal without--force) + MCPcgis_init_ontologyreturning YAML text read-only — no file writes from MCP (avoids the S2083 taint class)docs/ontology/patterns.yamlby a staleness testquery/ontology_init.pymodule, append-only CLI/MCP blocks (parallel-session contract)Test Plan
🤖 Generated with Claude Code