Skip to content

Add discovery pipeline orchestrator foundations (#126)#141

Merged
Dimwiddle merged 2 commits intofeature/discovery-layerfrom
feature/discovery-layer-126-pipeline-orchestrator
Mar 6, 2026
Merged

Add discovery pipeline orchestrator foundations (#126)#141
Dimwiddle merged 2 commits intofeature/discovery-layerfrom
feature/discovery-layer-126-pipeline-orchestrator

Conversation

@Dimwiddle
Copy link
Contributor

Description

Implements discovery orchestration infrastructure for issue #126:

  • Added DiscoveryConfig with from_pyproject()/default handling
  • Added FrameworkDetector for Python + TS/JS framework detection
  • Added immutable MinerContext
  • Added DiscoveryPipeline with registration guards, language filtering, per-miner error isolation, and report aggregation
  • Added build_default_pipeline() wiring (FileIndex, FrameworkDetector, MinerContext) and a minimal built-in language-agnostic miner (readme_overview) for baseline non-empty integration runs
  • Added tests for config, framework detection, pipeline behavior, and integration on this repo
  • Updated discovery feature spec scenarios/criteria for orchestration behaviors

Type of Change

  • New feature
  • Tests
  • Documentation update

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that this feature works
  • New and existing unit tests pass locally with my changes

Testing

  • make lint-fix
  • make lint ⚠️ fails due pre-existing mypy errors in src/specleft/commands/features.py (lines 818, 1039, 1040), unrelated to this PR
  • make test ✅ (473 passed)

Related Issues

Closes #126

@Dimwiddle Dimwiddle merged commit b588153 into feature/discovery-layer Mar 6, 2026
1 check passed
@Dimwiddle Dimwiddle deleted the feature/discovery-layer-126-pipeline-orchestrator branch March 6, 2026 20:04
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.

1 participant