Skip to content

feat: monorepo phase 3 drift-engine capability package (ADR-100)#664

Open
mick-gsk wants to merge 1 commit into
mainfrom
split/576-monorepo-phase3-drift-engine
Open

feat: monorepo phase 3 drift-engine capability package (ADR-100)#664
mick-gsk wants to merge 1 commit into
mainfrom
split/576-monorepo-phase3-drift-engine

Conversation

@mick-gsk
Copy link
Copy Markdown
Owner

@mick-gsk mick-gsk commented May 3, 2026

Split from #576 (ADR-100 monorepo migration). Part of the PR decomposition into atomic, reviewable units.

@sauremilk
feat: monorepo phase 3 -- drift-engine capability package (signals/sc…
b7a2b85 
…oring/ingestion/pipeline/analyzer, ADR-100)

(cherry picked from commit f356465)
Copilot AI review requested due to automatic review settings May 3, 2026 17:02
Copilot AI reviewed May 3, 2026
View reviewed changes
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

@mick-gsk
Copy link
Copy Markdown
Owner Author

mick-gsk commented May 3, 2026

Superseded by atomic sub-PRs #708 (engine package), #709 (signal stubs), #710 (ingestion/scoring stubs), #711 (tests), #712 (scripts/docs). Original PR too large (134 files) to review atomically.

@mick-gsk mick-gsk closed this May 3, 2026
github-advanced-security[bot]
github-advanced-security AI found potential problems May 3, 2026
View reviewed changes
Copy link
Copy Markdown

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

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

CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@github-actions github-actions Bot added documentation Improvements or additions to documentation signals Changes to signal detection logic tests Test coverage or fixture improvements ci Changes to CI/CD workflows or tooling scoring Changes to composite scoring or weights ingestion Changes to file discovery or AST parsing labels May 3, 2026
@mick-gsk mick-gsk reopened this May 4, 2026
@github-actions github-actions Bot added agent-review-requested Agent review was requested automatically lane/standard Fixes and features — standard review path lane/high-risk Signals/scoring/architecture — full gates + audit required labels May 4, 2026
@mick-gsk mick-gsk added release:feature Include this PR under Features in release notes size/XL Diff ≥ 500 lines — consider splitting labels May 4, 2026
@github-actions github-actions Bot added the has-conflicts PR has merge conflicts with the base branch label May 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

+1 more reviewer

@github-advanced-security github-advanced-security[bot] github-advanced-security[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

agent-review-requested Agent review was requested automatically ci Changes to CI/CD workflows or tooling documentation Improvements or additions to documentation has-conflicts PR has merge conflicts with the base branch ingestion Changes to file discovery or AST parsing lane/high-risk Signals/scoring/architecture — full gates + audit required lane/standard Fixes and features — standard review path release:feature Include this PR under Features in release notes scoring Changes to composite scoring or weights signals Changes to signal detection logic size/XL Diff ≥ 500 lines — consider splitting tests Test coverage or fixture improvements

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mick-gsk @github-advanced-security @sauremilk