Skip to content

OpenConceptLab/ocl_issues#2471 | Generalize bridge matching#10

Merged
snyaggarwal merged 1 commit intomainfrom
feature/generalize-bridge-matching
Apr 14, 2026
Merged

OpenConceptLab/ocl_issues#2471 | Generalize bridge matching#10
snyaggarwal merged 1 commit intomainfrom
feature/generalize-bridge-matching

Conversation

@paynejd
Copy link
Copy Markdown
Member

@paynejd paynejd commented Apr 14, 2026

Summary

  • Parameterizes bridge matching to support custom bridge sources beyond CIEL
  • Replaces hardcoded CIEL URLs with dynamic target_repo_url from algo config (defaults to CIEL)
  • Adds "Bridge Source URL" text field in algorithm settings panel, visible only to core_user group members
  • Accepts both ocl-bridge and ocl-ciel-bridge algo types for backward compatibility
  • All 5 files with CIEL-specific references updated

Files changed

  • MapProject.jsx — State, bridge detection, mapped sources fetch, BridgeMatch props, ~10 string refs
  • MultiAlgoSelector.jsx — Bridge icon detection, bridge source URL field (core_user gated)
  • Concept.jsx — Bridge algo detection
  • Candidates.jsx — Bridge flag for candidate display
  • ConfigurationForm.jsx — Bridge algo type check, isCoreUser prop threading

Related PRs

Part of OpenConceptLab/ocl_issues#2471

Test plan

  • Existing CIEL bridge projects work unchanged
  • Core user can expand bridge algo settings and see Bridge Source URL field
  • Non-core user does NOT see Bridge Source URL field
  • Entering a custom bridge source URL and running auto-match returns bridge-formatted results
  • Multi-algo eval (bridge + semantic) reranks and exports correctly
  • Bridge results display with cascade mapping chain visualization

🤖 Generated with Claude Code

…ort custom bridge sources

- Replace hardcoded CIEL URLs with dynamic bridge source URL from algo config
- Replace CIELMappedSources state with bridgeMappedSources map keyed by URL
- Accept both 'ocl-bridge' and 'ocl-ciel-bridge' algo types for backward compat
- Add Bridge Source URL field in MultiAlgoSelector settings (core_user only)
- Use .includes('bridge') for bridge detection in Concept.jsx and Candidates.jsx
- Thread isCoreUser prop through ConfigurationForm to MultiAlgoSelector

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@snyaggarwal snyaggarwal merged commit 22f56db into main Apr 14, 2026
1 check passed
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