drift-analyzer 2.7.0__tar.gz → 2.7.2__tar.gz

drift_analyzer-2.7.2/.github/ISSUE_TEMPLATE/real_repo_evaluation.yml

@@ -0,0 +1,103 @@
+name: Real-Repo Evaluation
+description: Share your experience running drift on a real-world repository
+title: "[Eval] "
+labels: ["real-repo-evaluation"]
+body:
+  - type: markdown
+    attributes:
+      value: |
+        Thanks for trying drift on a real repository! Your feedback helps us improve signal quality and first-run experience.
+
+        **Tip:** Run `drift analyze --report report.md` to generate a shareable Markdown report, then paste it below or attach the file.
+
+  - type: input
+    id: repo
+    attributes:
+      label: Repository
+      description: Link to the repository or its name (public or description of private repo)
+      placeholder: "e.g. https://github.com/org/repo or 'internal Django monolith, ~50k LOC'"
+    validations:
+      required: true
+
+  - type: input
+    id: drift_version
+    attributes:
+      label: Drift version
+      description: "Output of `drift --version`"
+      placeholder: "e.g. 2.5.1"
+    validations:
+      required: true
+
+  - type: dropdown
+    id: overall_impression
+    attributes:
+      label: Overall impression
+      description: How useful were drift's findings?
+      options:
+        - "Very useful — found real issues I should address"
+        - "Somewhat useful — some good findings mixed with noise"
+        - "Neutral — nothing surprising, nothing wrong"
+        - "Not useful — mostly false positives or irrelevant"
+        - "Confusing — output was hard to interpret"
+    validations:
+      required: true
+
+  - type: textarea
+    id: setup_experience
+    attributes:
+      label: Setup experience
+      description: "How was the installation and first run? Any friction?"
+      placeholder: |
+        e.g. "pip install worked cleanly, but first run timed out on our monorepo"
+        or "Had trouble with shallow clones in CI"
+    validations:
+      required: false
+
+  - type: textarea
+    id: useful_findings
+    attributes:
+      label: Most useful findings
+      description: "Which findings resonated? Were there any 'aha' moments?"
+      placeholder: |
+        e.g. "PFS correctly identified our duplicated validation logic across 3 modules"
+    validations:
+      required: false
+
+  - type: textarea
+    id: false_positives
+    attributes:
+      label: False positives or noise
+      description: "Any findings that were clearly wrong or unhelpful? Include signal ID + file if possible."
+      placeholder: |
+        e.g. "DIA flagged our generated API docs — these are auto-generated and intentionally diverge"
+    validations:
+      required: false
+
+  - type: textarea
+    id: missing_findings
+    attributes:
+      label: Things drift missed
+      description: "Any known issues in your codebase that drift did not detect?"
+      placeholder: |
+        e.g. "We have significant layering violations between api/ and core/ that weren't flagged"
+    validations:
+      required: false
+
+  - type: textarea
+    id: report
+    attributes:
+      label: Drift report (optional)
+      description: |
+        Paste the output of `drift analyze --report report.md` or attach the file.
+        Wrap in a details tag if it's long.
+      render: markdown
+    validations:
+      required: false
+
+  - type: textarea
+    id: suggestions
+    attributes:
+      label: Suggestions
+      description: "What would make drift more useful for your workflow?"
+    validations:
+      required: false

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/.github/copilot-instructions.md

@@ -199,14 +199,16 @@ Wenn ein Agent Drift-Findings über MCP-Tools beheben soll, **muss** dieser Abla
 
 1. **`drift_session_start(path=".", autopilot=true)`** — ein Aufruf statt vier (bündelt validate + brief + scan + fix_plan)
 2. **`drift_nudge(session_id=..., changed_files=...)`** — nach jeder Dateiänderung als schneller Inner-Loop (~0.2 s statt ~3 s für scan)
-3. **`drift_fix_plan(session_id=..., max_tasks=1)`** — nächsten Task holen (immer `max_tasks=1`)
-4. **`drift_diff(session_id=..., uncommitted=true)`** — nur einmal am Ende als Abschluss-Verifikation
+3. **Test-Checkpoint nach `nudge`** — gezielte Tests per Pfad-Matrix ausführen (Tabelle in `.github/prompts/drift-fix-loop.prompt.md`, Schritt 3b); bei Fehlschlag Entscheidungsbaum anwenden: Test anpassen (Implementation-Details) oder Fix reverten (Vertrags-Regression) — kein Hard-Block
+4. **`drift_fix_plan(session_id=..., max_tasks=1)`** — nächsten Task holen (immer `max_tasks=1`)
+5. **`drift_diff(session_id=..., uncommitted=true)`** — nur einmal am Ende als Abschluss-Verifikation
 
 **Verboten im Fix-Loop:**
 - `drift_scan` nach jeder Dateiänderung (zu teuer, nutze `nudge`)
 - `session_start` ohne `autopilot=true` (verschenkt 4 Roundtrips)
 - `fix_plan` ohne `max_tasks=1` (unnötig große Responses)
 - Tool-Aufrufe ohne `session_id` (verliert Kontext)
+- Commit ohne Test-Checkpoint (Tests fallen sonst erst bei Gate 8 CI auf)
 
 **Immer:** `agent_instruction` und `next_tool_call` aus Responses befolgen.
 
@@ -232,8 +234,8 @@ Vollständiger Developer Guide: **[DEVELOPER.md](../DEVELOPER.md)**
 
 ```
 ingestion/ → signals/ → scoring/ → output/
-  AST + Git     23 Detektoren  Score+Severity   Rich/JSON/SARIF
-              (15 scoring-aktiv, 8 report-only)
+  AST + Git     24 Detektoren  Score+Severity   Rich/JSON/SARIF
+              (15 scoring-aktiv, 9 report-only)
 ```
 
 ### Wichtigste Kommandos
@@ -288,7 +290,7 @@ DRIFT_SKIP_HOOKS=1 git push          # ALLE Gates (äußerster Notfall)
 
 | Pfad | Inhalt |
 |------|--------|
-| `src/drift/signals/` | 23 Signale — 15 scoring-aktiv (PFS, AVS, MDS, EDS, TVS, SMS, DIA, BEM, TPD, GCD, NBV, BAT, ECM, COD, CCC) + 8 report-only (TSA, CXS, FOE, CIR, DCA, MAZ, ISD, HSC) |
+| `src/drift/signals/` | 24 Signale — 15 scoring-aktiv (PFS, AVS, MDS, EDS, TVS, SMS, DIA, BEM, TPD, GCD, NBV, BAT, ECM, COD, CCC) + 9 report-only (TSA, CXS, FOE, CIR, DCA, MAZ, ISD, HSC, PHR) |
 | `src/drift/ingestion/` | AST-Parsing, Git-History, File-Discovery |
 | `src/drift/scoring/` | Composite-Score, Module-Scores, Severity |
 | `src/drift/output/` | Rich-Terminal, JSON, SARIF |

drift_analyzer-2.7.2/.github/instructions/drift-context-routing.instructions.md

@@ -0,0 +1,40 @@
+---
+applyTo: "**"
+description: "Kontext-Routing — Zuordnung von Aufgabentypen zu relevanten Instructions und Skills. Hilft Agenten, bei großem Instruction-Volumen die richtigen Regeln zu priorisieren."
+---
+
+# Kontext-Routing — Welche Regeln gelten wann?
+
+> **Zweck:** Nicht alle Instructions und Skills sind für jede Aufgabe gleich relevant.
+> Diese Datei hilft dem Agenten, den Fokus auf die wichtigsten Regeln zu legen.
+
+## Routing-Tabelle
+
+| Aufgabentyp | Primäre Instructions | Relevante Skills | Primärer Fokus |
+|-------------|---------------------|-----------------|----------------|
+| **Signal-Arbeit** (neues Signal, Heuristik-Änderung) | drift-policy, drift-push-gates (Gate 2+7) | drift-signal-development-full-lifecycle, drift-risk-audit-artifact-updates, drift-adr-workflow | ADR-Pflicht, Audit-Pflicht, Precision/Recall |
+| **Scoring/Gewichte** | drift-policy (§6 Priorisierung) | drift-adr-workflow | ADR-Pflicht, Benchmark-Evidence |
+| **Output-Format** | drift-policy, drift-push-gates (Gate 7) | drift-adr-workflow, drift-risk-audit-artifact-updates | ADR-Pflicht, STRIDE-Update |
+| **Bugfix** (`fix:`) | drift-policy, drift-push-gates (Gate 3+8) | drift-commit-push | CHANGELOG, Tests, kein ADR nötig |
+| **Feature** (`feat:`) | drift-policy, drift-push-gates (Gate 2+3) | drift-commit-push | Feature-Evidence, Tests, CHANGELOG, STUDY.md |
+| **Refactoring** | drift-policy, drift-quality-workflow | drift-commit-push | Kein ADR nötig, Tests grün |
+| **Prompt/Instruction** | drift-policy | drift-agent-prompt-authoring | Shared Partials, Modellunabhängigkeit |
+| **PR-Review** | drift-quality-workflow (Stufe 4) | drift-pr-review | Review-Checkliste (`.github/prompts/_partials/review-checkliste.md`) |
+| **Release/Push** | drift-push-gates (alle Gates) | drift-commit-push, drift-release | Alle Gates prüfen, `make check` |
+| **Fixture/Test** | drift-policy | drift-ground-truth-fixture-development | Precision/Recall, Ground Truth |
+| **Dokumentation** | drift-policy (§13 Befund-Qualität) | — | Keine Gate-Pflicht außer Gate 8 |
+
+## Priorisierungsregel
+
+Wenn mehrere Instructions geladen sind, gilt:
+1. **drift-policy** hat immer Vorrang (PFLICHT-GATE zuerst)
+2. **drift-push-gates** nur relevant bei Commit/Push-Vorbereitung
+3. **drift-quality-workflow** nur relevant bei nicht-trivialen Änderungen
+4. **Release-Instructions** nur relevant bei `src/drift/**`-Änderungen
+
+## Kontextreduktion
+
+Bei einfachen Aufgaben (Doku-Fix, Typo, triviales Refactoring) reicht:
+- Policy-Gate ausgeben
+- Relevante Gates prüfen (meist nur Gate 8: `make check`)
+- Kein vollständiger Quality-Workflow nötig

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/.github/instructions/drift-quality-workflow.instructions.md

@@ -61,6 +61,8 @@ Keine „quick hacks" ohne Begründungsstruktur.
 
 Nach jedem pushbaren Zwischenstand wird ein separater BugBot-Review-Schritt ausgeführt.
 
+**Strukturierte Review-Checkliste verwenden:** Der Reviewer arbeitet die Checkliste unter `.github/prompts/_partials/review-checkliste.md` Punkt für Punkt ab und dokumentiert pro Punkt Ja / Nein / N/A mit Kurzbegründung. Kein Punkt darf übersprungen werden.
+
 **Dieser Review bewertet isoliert und streng:**
 - Korrektheit der Änderung
 - Risiko und Edge Cases

drift_analyzer-2.7.2/.github/instructions/drift-release-mandatory.instructions.md

@@ -0,0 +1,19 @@
+---
+applyTo: "src/drift/**"
+description: "Releases are automated via python-semantic-release in CI. Agents must use conventional commits (feat:/fix:/BREAKING:) — no manual release command needed."
+---
+
+# Release Automation
+
+> **Vollständige Dokumentation:** siehe `drift-release-automation.instructions.md` (gleicher Scope).
+
+**Kurzregel:** Conventional Commits (`feat:`, `fix:`, `BREAKING:`) verwenden — CI übernimmt Version, Tag, Release, PyPI.
+Kein manueller Release-Befehl nötig. Fallback: `python scripts/release_automation.py --full-release`.
+python scripts/release_automation.py --full-release
+```
+
+## See Also
+
+- Release Workflow: `.github/workflows/release.yml`
+- PSR Configuration: `pyproject.toml` → `[tool.semantic_release]`
+- Detailed Skill: `.github/skills/drift-release/SKILL.md`

drift_analyzer-2.7.2/.github/prompts/_partials/review-checkliste.md

@@ -0,0 +1,80 @@
+# Review-Checkliste (Shared Reference)
+
+> **Single Source of Truth** — Alle adversarialen Reviews verwenden diese Checkliste.
+> Die Checkliste ersetzt keine Eigenleistung des Reviewers, sie strukturiert sie.
+
+## Anwendung
+
+Der Reviewer arbeitet die Checkliste Punkt für Punkt ab und dokumentiert pro Punkt:
+- **Ja** / **Nein** / **N/A** (mit Kurzbegründung bei Nein oder N/A)
+
+Kein Punkt darf übersprungen werden. Bei Unsicherheit: `Nein` mit Begründung.
+
+---
+
+## 1. Policy-Konformität
+
+- [ ] **Policy-Gate korrekt?** — Wurde das PFLICHT-GATE sichtbar ausgegeben, und ist die Begründung plausibel (nicht rituell)?
+- [ ] **Zulassungskriterium nachvollziehbar?** — Ist das benannte Kriterium (Unsicherheit / Signal / Glaubwürdigkeit / Handlungsfähigkeit / Trend / Einführbarkeit) tatsächlich adressiert?
+- [ ] **Ausschlusskriterium geprüft?** — Erzeugt die Änderung ausschließlich mehr Ausgabe/Komplexität/Features ohne benennbaren Nutzen?
+- [ ] **Priorisierung eingehalten?** — Verdrängt die Änderung eine höherpriorisierte Aufgabe (§6)?
+
+## 2. Scope und Boundaries
+
+- [ ] **Scope eingehalten?** — Wurden nur Dateien geändert, die zur Aufgabe gehören?
+- [ ] **Out-of-Scope unberührt?** — Wurden keine Dateien modifiziert, die explizit oder implizit außerhalb des Auftrags liegen?
+- [ ] **Keine ungefragt hinzugefügten Features?** — Enthält die Änderung nur das Angefragte, keine "Verbesserungen" oder Refactorings als Beifang?
+
+## 3. Architektur und ADR-Pflicht
+
+- [ ] **ADR erforderlich?** — Betrifft die Änderung Signale, Scoring, Output oder Architektur-Boundaries? Falls ja: ADR unter `decisions/` vorhanden (Status `proposed` reicht)?
+- [ ] **Architektur-Layer korrekt?** — Respektiert die Änderung den Datenfluss `ingestion → signals → scoring → output`?
+- [ ] **Keine unzulässigen Layer-Übergriffe?** — Greift kein Modul auf eine Schicht zu, zu der es laut Architektur keinen Zugang haben sollte?
+
+## 4. Audit-Pflicht (Policy §18)
+
+- [ ] **Betrifft die Änderung `signals/`, `ingestion/` oder `output/`?** — Falls nein: N/A.
+- [ ] **Audit-Artefakte aktualisiert?** — Wurde mindestens das richtige Artefakt inhaltlich (nicht nur kosmetisch) aktualisiert?
+  - Signal-Änderung → `fmea_matrix.md` + `fault_trees.md` + `risk_register.md`
+  - Input/Output-Pfad → `stride_threat_model.md` + `risk_register.md`
+  - Precision/Recall Δ > 5% → `fmea_matrix.md` (RPNs) + `risk_register.md`
+- [ ] **Alle vier Audit-Artefakte existieren noch?** — Kein Löschrisiko?
+
+## 5. Befund-Qualität (§13)
+
+Nur relevant, wenn die Änderung Analyseergebnisse / Befunde betrifft.
+
+- [ ] **Technische Nachvollziehbarkeit?** — Kann ein Dritter den Befund anhand des Codes rekonstruieren?
+- [ ] **Reproduzierbarkeit?** — Liefert dieselbe Eingabe denselben Befund?
+- [ ] **Eindeutige Ursachenzuordnung?** — Ist klar, welche Code-Stelle den Befund auslöst?
+- [ ] **Nachvollziehbare Begründung?** — Steht eine Erklärung, warum dies ein Problem ist?
+- [ ] **Erkennbare nächste Maßnahme?** — Weiß der Empfänger, was zu tun ist?
+
+## 6. Code-Qualität
+
+- [ ] **Docstrings vorhanden?** — Jede neue öffentliche Funktion unter `src/drift/` hat einen Docstring?
+- [ ] **Keine offensichtlichen Bugs?** — Edge Cases, Off-by-One, Null-Handling geprüft?
+- [ ] **Keine Sicherheitslücken?** — OWASP Top 10 beachtet (Path Traversal, Injection, etc.)?
+- [ ] **Typisierung konsistent?** — mypy-konform, keine unannotierten öffentlichen Signaturen?
+
+## 7. Tests
+
+- [ ] **Tests vorhanden?** — Gibt es Tests für das neue/geänderte Verhalten?
+- [ ] **Regressionstests?** — Bestehendes Verhalten durch Tests abgesichert?
+- [ ] **Edge Cases abgedeckt?** — Leere Eingaben, Grenzwerte, Fehlerpfade?
+- [ ] **Keine duplizierten Testnamen?** — Kein `def test_...`-Duplikat im selben Modul?
+
+## 8. Commit und Gates
+
+- [ ] **Conventional Commit korrekt?** — `feat:`, `fix:`, `refactor:`, `docs:`, `test:`, `chore:` passend gewählt?
+- [ ] **CHANGELOG aktualisiert?** — Bei `feat:` oder `fix:`: Eintrag in `CHANGELOG.md` vorhanden?
+- [ ] **Feature-Evidence vorhanden?** — Bei `feat:`: Tests + `benchmark_results/vX.Y.Z_*.json` + `docs/STUDY.md`?
+- [ ] **Lockfile synchron?** — Bei `pyproject.toml`-Änderung: `uv.lock` aktualisiert?
+
+## 9. Gesamtbewertung
+
+- [ ] **Bekannte Restrisiken benannt?** — Sind Trade-offs oder offene Punkte explizit dokumentiert (nicht verschwiegen)?
+- [ ] **Freigabestatus bestimmt:**
+  - `❌ NICHT BEREIT` — offene kritische Issues
+  - `⚠️ BEREIT ZUR MENSCHLICHEN PRÜFUNG` — Checks grün, aber Risiken sichtbar
+  - `✅ BEREIT ZUM MERGE` — vollständig grün, alle Gates erfüllt

drift_analyzer-2.7.2/.github/prompts/drift-fix-loop.prompt.md

@@ -0,0 +1,195 @@
+---
+name: "Drift Fix-Loop"
+description: "Optimierter Agent-Workflow zum Beheben von Drift-Findings über MCP-Sessions. Nutzt session_start(autopilot=true) + nudge als Inner-Loop für minimale Roundtrips."
+---
+
+# Drift Fix-Loop
+
+Dieser Prompt beschreibt den optimalen Ablauf, um Drift-Findings in einem Workspace über MCP-Tools effizient zu beheben. Der Workflow minimiert Roundtrips und nutzt die vorhandene Session-Infrastruktur vollständig aus.
+
+> **Pflicht:** Vor Ausführung dieses Prompts das Drift Policy Gate durchlaufen
+> (siehe `.github/prompts/_partials/konventionen.md` und `.github/instructions/drift-policy.instructions.md`).
+
+## Relevante Referenzen
+
+- **Instruction:** `.github/instructions/drift-policy.instructions.md`
+- **Push Gates:** `.github/instructions/drift-push-gates.instructions.md`
+- **Skill:** `.github/skills/drift-effective-usage/SKILL.md`
+- **Skill:** `.github/skills/drift-commit-push/SKILL.md`
+- **MCP-Server:** `src/drift/mcp_server.py`
+- **Session-Management:** `src/drift/session.py`
+
+## Warum dieser Workflow
+
+Ein naiver Agent-Loop ruft pro Finding: `validate → scan → fix_plan → [edit] → scan → diff` auf.
+Jeder `scan`-Aufruf führt eine vollständige Repo-Analyse durch (~1–5 s). Bei mehreren Findings multipliziert sich die Wartezeit und die Roundtrips.
+
+**Dieser Workflow reduziert das auf:**
+1. Einen einzigen `session_start(autopilot=true)`-Aufruf (bündelt validate + brief + scan + fix_plan)
+2. `nudge` als schnellen Inner-Loop nach jeder Dateiänderung (~0.2 s statt ~3 s für scan)
+3. Einen einzigen `drift_diff` als Abschluss-Verifikation
+
+## Ablauf
+
+### Schritt 1 — Session starten mit Autopilot
+
+```
+drift_session_start(
+    path=".",
+    autopilot=true
+)
+```
+
+Dieser eine Aufruf führt automatisch `validate → brief → scan → fix_plan` aus und gibt die kombinierten Ergebnisse zurück. Die Session-ID wird in allen Folge-Aufrufen wiederverwendet.
+
+**Wichtig:** Die `session_id` aus der Antwort merken und an **jeden** weiteren Tool-Aufruf übergeben.
+
+### Schritt 2 — Ersten Task aus fix_plan bearbeiten
+
+Die Autopilot-Antwort enthält bereits den `fix_plan`. Den **ersten** Task nehmen und umsetzen.
+
+**Regel:** Immer nur einen Task gleichzeitig. Nicht mehrere Findings in einem Schritt mischen.
+
+### Schritt 3 — Nach jeder Dateiänderung: nudge
+
+```
+drift_nudge(
+    session_id="<session_id>",
+    changed_files="<pfad/zur/geänderten/datei.py>"
+)
+```
+
+`nudge` gibt schnelles Richtungsfeedback:
+- `direction`: improving / stable / degrading
+- `safe_to_commit`: true / false
+- `confidence`: pro Signal
+
+**Wenn `direction=degrading`:** Änderung rückgängig machen, anders lösen.
+**Wenn `safe_to_commit=true`:** Weiter mit Test-Checkpoint (Schritt 3b).
+
+### Schritt 3b — Test-Checkpoint nach jeder Änderung
+
+Nachdem `nudge` kein Degrading meldet, **gezielte Tests** für die geänderte Datei ausführen.
+
+**Pfad-zu-Test-Mapping** (kürzeste Match-Regel, oben zuerst):
+
+| Geänderte Datei (Pattern) | Gezielte Tests |
+|---|---|
+| `src/drift/signals/architecture_violation*` | `pytest tests/test_avs_*.py -q --tb=short` |
+| `src/drift/signals/doc_impl_drift*` | `pytest tests/test_dia_*.py -q --tb=short` |
+| `src/drift/signals/explainability_deficit*` | `pytest tests/test_eds_*.py -q --tb=short` |
+| `src/drift/signals/mutant_duplicates*` | `pytest tests/test_mutant_duplicates*.py -q --tb=short` |
+| `src/drift/signals/dead_code_accumulation*` | `pytest tests/test_dead_code*.py -q --tb=short` |
+| `src/drift/signals/pattern_fragmentation*` | `pytest tests/test_pattern_fragmentation*.py -q --tb=short` |
+| `src/drift/signals/naming_contract*` | `pytest tests/test_naming_contract*.py -q --tb=short` |
+| `src/drift/signals/test_polarity_deficit*` | `pytest tests/test_test_polarity_deficit*.py -q --tb=short` |
+| `src/drift/signals/cognitive_complexity*` | `pytest tests/test_cognitive_complexity*.py -q --tb=short` |
+| `src/drift/signals/circular_import*` | `pytest tests/test_circular_import*.py -q --tb=short` |
+| `src/drift/signals/guard_clause*` | `pytest tests/test_guard_clause*.py -q --tb=short` |
+| `src/drift/signals/insecure_default*` | `pytest tests/test_insecure_default*.py -q --tb=short` |
+| `src/drift/signals/missing_authorization*` | `pytest tests/test_missing_authorization*.py -q --tb=short` |
+| `src/drift/signals/hardcoded_secret*` | `pytest tests/test_hardcoded_secret*.py -q --tb=short` |
+| `src/drift/signals/exception_contract*` | `pytest tests/test_exception_contract*.py -q --tb=short` |
+| `src/drift/signals/fan_out_explosion*` | `pytest tests/test_fan_out_explosion*.py -q --tb=short` |
+| `src/drift/signals/cohesion_deficit*` | `pytest tests/test_cohesion_deficit*.py -q --tb=short` |
+| `src/drift/signals/bypass_accumulation*` | `pytest tests/test_bypass_accumulation*.py -q --tb=short` |
+| `src/drift/signals/*` (andere) | `pytest tests/test_precision_recall.py tests/test_mirofish_signal_improvements.py -q --tb=short` |
+| `src/drift/api.py` | `pytest tests/test_brief.py tests/test_integration.py tests/test_incremental.py tests/test_fix_actionability.py tests/test_nudge.py -q --tb=short` |
+| `src/drift/mcp_server.py` | `pytest tests/test_mcp_copilot.py tests/test_mcp_hardening.py tests/test_tool_metadata.py tests/test_negative_context_export.py -q --tb=short` |
+| `src/drift/output/*` | `pytest tests/test_json_output.py tests/test_csv_output.py tests/test_sarif_contract.py tests/test_output_golden.py tests/test_agent_tasks.py -q --tb=short` |
+| `src/drift/ingestion/*` | `pytest tests/test_ast_parser.py tests/test_file_discovery.py tests/test_scope_resolver.py tests/test_typescript_parser.py -q --tb=short` |
+| `src/drift/config.py` | `pytest tests/test_config.py tests/test_config_validate.py tests/test_model_consistency.py -q --tb=short` |
+| `src/drift/commands/*` | `pytest tests/test_self_command.py tests/test_patterns_command.py tests/test_ci_reality.py -q --tb=short` |
+| `src/drift/session.py` | `pytest tests/test_session.py -q --tb=short` |
+| `src/drift/incremental.py` | `pytest tests/test_incremental.py tests/test_nudge.py -q --tb=short` |
+| Fallback (kein Treffer) | `pytest tests/ -q --tb=short --ignore=tests/test_smoke_real_repos.py --maxfail=5` |
+
+**Bei Testfehlschlag — Entscheidungsbaum:**
+
+| Fehlerbild | Ursache | Reaktion |
+|---|---|---|
+| `AttributeError: has no attribute X` | Test prüft Implementation-Details | **Test anpassen** |
+| `TypeError: N arguments expected` | Signatur wurde geändert | **Test anpassen** |
+| Test erwartet altes Drift-Pattern, das absichtlich entfernt wurde | Finding war Ziel des Fixes | **Test anpassen** |
+| `AssertionError` auf dokumentiertes Public-API-Verhalten | Vertrag verletzt | **Production-Fix überdenken oder reverten** |
+| `AssertionError` auf Rückgabewert eines öffentlichen Contracts | Semantische Regression | **Production-Fix überdenken oder reverten** |
+
+**Kein Hard-Block:** Der Workflow wird nicht abgebrochen. Der Agent wendet den Entscheidungsbaum an und setzt mit dem bereinigten Zustand fort.
+
+### Schritt 4 — Nächsten Task holen (falls nötig)
+
+Wenn weitere Findings offen sind:
+
+```
+drift_fix_plan(
+    session_id="<session_id>",
+    max_tasks=1
+)
+```
+
+**Immer `max_tasks=1` verwenden.** Das reduziert die Response-Größe und den Parsing-Overhead erheblich.
+
+Dann zurück zu Schritt 2.
+
+### Schritt 5 — Abschluss-Verifikation
+
+Erst wenn alle Tasks bearbeitet sind:
+
+```
+drift_diff(
+    session_id="<session_id>",
+    uncommitted=true
+)
+```
+
+Prüfen:
+- `resolved_count` > 0
+- `new_count` == 0
+- Keine Regressionen
+
+## Zusammenfassung als Ablaufdiagramm
+
+```
+session_start(autopilot=true)       ← 1 Aufruf statt 4
+    ↓
+fix_plan (in Autopilot enthalten)
+    ↓
+┌──────────────────────────────────────────┐
+│  Task N bearbeiten                       │
+│  Datei editieren                         │
+│  nudge(changed_files=...)  ← ~0.2 s      │
+│  direction prüfen                        │
+│  ggf. korrigieren                        │
+│  pytest <gezielte Tests> --tb=short  ← 3b│
+│  bei rot: Test anpassen oder reverten    │
+└─────────┬────────────────────────────────┘
+          │ safe_to_commit + Tests grün?
+          ↓
+    fix_plan(max_tasks=1)    ← nächster Task
+          │ keine Tasks mehr?
+          ↓
+    drift_diff(uncommitted)  ← Abschluss-Verifikation
+          ↓
+    Commit vorbereiten
+```
+
+## Anti-Patterns: Was NICHT tun
+
+| Anti-Pattern | Problem | Stattdessen |
+|---|---|---|
+| `drift_scan` nach jeder Dateiänderung | Volle Repo-Analyse pro Edit (~3–5 s) | `drift_nudge` verwenden (~0.2 s) |
+| `session_start` ohne `autopilot=true` | 4 separate Roundtrips für validate/brief/scan/fix_plan | `autopilot=true` setzen |
+| `fix_plan(max_tasks=5)` oder mehr | Große Response, Agent parst unnötig viel | `max_tasks=1` im Loop verwenden (initiale Übersicht wird von autopilot oder scan-Guidance gesteuert) |
+| Kein `session_id` weitergeben | Jeder Aufruf verliert Kontext, kein Scope-Carry-Over | Immer `session_id` übergeben |
+| `drift_diff` nach jedem Edit | Teurer als nötig für Zwischenfeedback | `nudge` als Inner-Loop, `diff` nur am Ende |
+| Mehrere Findings gleichzeitig fixen | Unklar welche Änderung welchen Effekt hat | Immer ein Finding pro Iteration |
+
+## Verhalten bei `agent_instruction` in Responses
+
+Drift-MCP-Tools geben in jeder Antwort ein Feld `agent_instruction` zurück. **Dieses Feld befolgen.** Es enthält den empfohlenen nächsten Schritt, der zum Session-Zustand passt.
+
+Zusätzlich gibt `next_tool_call` den konkreten nächsten Tool-Aufruf mit Parametern vor. Wenn vorhanden, diesen bevorzugt verwenden.
+
+## Artefakte
+
+Dieser Prompt erzeugt keine separaten Artefakte. Die Änderungen sind die Code-Fixes selbst.

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/.github/skills/drift-effective-usage/SKILL.md

@@ -181,6 +181,40 @@ Key rules:
 - Follow `agent_instruction` and `next_tool_call` fields in responses
 - Run `drift_diff` only as final verification, not after every edit
 
+### Test-Aware Fix-Loop
+
+After each `drift_nudge` call (when `direction` is not degrading), run targeted tests for the changed file **before** requesting the next task.
+
+**Why:** `fix_plan`, `nudge`, and `AgentTask` contain no test awareness. The `safe_to_commit` field only evaluates drift delta — not pytest results. Test failures are otherwise only caught at push time by CI Gate 8.
+
+**Path-to-test mapping (first match wins):**
+
+| Changed file pattern | Run these tests |
+|---|---|
+| `src/drift/signals/architecture_violation*` | `pytest tests/test_avs_*.py -q --tb=short` |
+| `src/drift/signals/doc_impl_drift*` | `pytest tests/test_dia_*.py -q --tb=short` |
+| `src/drift/signals/explainability_deficit*` | `pytest tests/test_eds_*.py -q --tb=short` |
+| `src/drift/signals/mutant_duplicates*` | `pytest tests/test_mutant_duplicates*.py -q --tb=short` |
+| `src/drift/signals/dead_code_accumulation*` | `pytest tests/test_dead_code*.py -q --tb=short` |
+| `src/drift/signals/pattern_fragmentation*` | `pytest tests/test_pattern_fragmentation*.py -q --tb=short` |
+| `src/drift/signals/*` (other) | `pytest tests/test_precision_recall.py tests/test_mirofish_signal_improvements.py -q --tb=short` |
+| `src/drift/api.py` | `pytest tests/test_brief.py tests/test_integration.py tests/test_nudge.py -q --tb=short` |
+| `src/drift/mcp_server.py` | `pytest tests/test_mcp_copilot.py tests/test_mcp_hardening.py tests/test_tool_metadata.py -q --tb=short` |
+| `src/drift/output/*` | `pytest tests/test_json_output.py tests/test_agent_tasks.py tests/test_output_golden.py -q --tb=short` |
+| `src/drift/ingestion/*` | `pytest tests/test_ast_parser.py tests/test_file_discovery.py -q --tb=short` |
+| `src/drift/config.py` | `pytest tests/test_config.py tests/test_config_validate.py -q --tb=short` |
+| `src/drift/session.py` | `pytest tests/test_session.py -q --tb=short` |
+| `src/drift/incremental.py` | `pytest tests/test_incremental.py tests/test_nudge.py -q --tb=short` |
+| Fallback | `pytest tests/ -q --tb=short --ignore=tests/test_smoke_real_repos.py --maxfail=5` |
+
+**On test failure — decision tree:**
+
+- `AttributeError: has no attribute X` or `TypeError: N arguments expected` → test checks implementation details → **adapt the test**
+- Test expects an old drift pattern that was intentionally removed → **adapt the test**
+- `AssertionError` on documented public API behavior or return value contract → **reconsider or revert the production fix**
+
+No hard block: the workflow continues after the decision is applied. Full mapping and decision rules: `.github/prompts/drift-fix-loop.prompt.md` (Schritt 3b).
+
 See `.github/prompts/drift-fix-loop.prompt.md` for the detailed workflow.
 
 ## False Positive Reduction Playbook

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/.github/skills/drift-pr-review/SKILL.md

@@ -15,6 +15,12 @@ Guide Copilot agents through a policy-conformant, evidence-based pull request re
 - Evaluating signal quality, scoring changes, or output format changes
 - Checking benchmark evidence for new features
 
+## Strukturierte Review-Checkliste
+
+**Pflicht-Referenz:** Die vollständige Review-Checkliste liegt unter `.github/prompts/_partials/review-checkliste.md`.
+Bei jedem adversarialen Review wird diese Checkliste Punkt für Punkt abgearbeitet.
+Der Reviewer dokumentiert pro Punkt **Ja / Nein / N/A** mit Kurzbegründung.
+
 ## Review Workflow
 
 ### Step 1: Policy Gate

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/.secrets.baseline

@@ -175,7 +175,7 @@
         "filename": "src\\drift\\models.py",
         "hashed_secret": "581318342217e0472a69081affd5f121ad664884",
         "is_verified": false,
-        "line_number": 49
+        "line_number": 55
       }
     ],
     "src\\drift\\output\\agent_tasks.py": [
@@ -308,5 +308,5 @@
       }
     ]
   },
-  "generated_at": "2026-04-05T12:58:01Z"
+  "generated_at": "2026-04-09T16:12:57Z"
 }

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/CHANGELOG.md

@@ -9,6 +9,7 @@
 
 ### Fixed
 
+- Enforce optional strict orchestration guards in MCP session workflows to avoid unsafe autonomous tool-chain progression when strict mode is enabled.
 - Add missing `response_profile` docstring entry in `drift_diff` MCP tool, fixing `test_catalog_parameters_have_descriptions` CI failure.
 - Exclude `.drift-baseline.json` from `detect-secrets` scan — fingerprint hex strings are analysis hashes, not secrets.
 - Deduplicate `include` glob patterns in `file_discovery.py` to prevent the same file from being processed and appended multiple times when it matches several include patterns; use lazy `glob()` iterator and reuse `relative_to()` result to reduce redundant I/O.
@@ -86,7 +87,15 @@
 - Map git-root-relative paths to repo-relative paths in `analyze_diff` and `parse_git_history` so `--repo` on nested subdirectories no longer leaks parent-repo file scope (#117).
 - `drift self` error guidance now suggests valid next actions (`drift scan`/`drift analyze`) instead of invalid `--repo` flag (#120).
 
-## [2.6.2] - 2026-04-09
+## [2.7.1] - 2026-04-09
+
+Short version: Align release metadata so release-discipline checks pass.
+
+### Changed
+
+- Align top changelog release marker with project version `2.7.1` in `pyproject.toml`.
+
+## [2.7.0] - 2026-04-09
 
 Short version: Signal-filtering for scan, cross-validation fields, and false-positive reductions across multiple signals.
 

{drift_analyzer-2.7.0 → drift_analyzer-2.7.2}/CITATION.cff

@@ -9,8 +9,8 @@ authors:
 repository-code: "https://github.com/mick-gsk/drift"
 url: "https://mick-gsk.github.io/drift/"
 license: MIT
-version: "2.6.2"
-date-released: "2026-04-08"
+version: "2.7.1"
+date-released: "2026-04-09"
 keywords:
   - static-analysis
   - architectural-drift