Skip to content

propagate ids_set amongst tools#7

Merged
Simon-McIntosh merged 3 commits into
iterorganization:mainfrom
Simon-McIntosh:main
Sep 3, 2025
Merged

propagate ids_set amongst tools#7
Simon-McIntosh merged 3 commits into
iterorganization:mainfrom
Simon-McIntosh:main

Conversation

@Simon-McIntosh

Copy link
Copy Markdown
Collaborator

No description provided.

@Simon-McIntosh
Simon-McIntosh merged commit 2a7abfb into iterorganization:main Sep 3, 2025
4 of 5 checks passed
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 56.52174% with 20 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
imas_mcp/tools/overview_tool.py 38.46% 8 Missing ⚠️
imas_mcp/cli.py 0.00% 7 Missing ⚠️
imas_mcp/search/document_store.py 84.61% 2 Missing ⚠️
imas_mcp/server.py 33.33% 2 Missing ⚠️
imas_mcp/resource_provider.py 66.66% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

Simon-McIntosh added a commit that referenced this pull request Dec 17, 2025
propagate ids_set amongst tools
Simon-McIntosh referenced this pull request in Simon-McIntosh/imas-codex Apr 22, 2026
Major rewrite of plan 36 (catalog quality refactor):

- Phase 0 (NEW): ISN postfix grammar inversion (rc22). Unifies vector
  and complex parent/part under ONE rule: modifiers strip right-to-left
  from closed reserved vocab. Resolves RD round 2 finding #1
  (vector/complex false-symmetry) by eliminating the asymmetry.
- Phase 2 (REWRITTEN): linking workflow rebuild targeting 11 bugs
  discovered via investigation + empirical graph query (all 927 names
  have 0 links, link_status=None). Fixes:
  B1 _sanitize_links dd: junk-strip (RD #4 BLOCKING)
  B2 contradictory LLM instructions (enrich_system vs style guide)
  B3 zero-link bootstrap failure (consequence of B1+B2)
  B4 no vector/complex sibling auto-populate (user req)
  B5 link_status never reaches unresolved
  B6 over-strict missing_reverse audit for parent/component
  B7 _sanitize_links drops name:X during bootstrap (chicken-egg)
  B8 no DD-path context in LLM prompt
  B9 no failed-link re-queue on new SN writes
  B10 # in junk-strip collides with URL fragments
  B11 DD cluster-peer link candidates unused
- Phase 1b export field-exclusion mechanism CREATED (did not exist).
- Phase 1c doc_resolution_status field added parallel to link_status.
- Phase 4b small cost-bound verifier run (equilibrium, 20 names, $2)
  with explicit gate criteria before opening taps.
- Phase 4d generalised sn bootstrap-derived-parents (vector+complex).
- RD round 2 non-grammar findings merged (#1b, #1d, #1e, #4, #5, #6,
  #7, #8a, #8b).

Status: PLANNING, RD round 3 pending.
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