devcouncil 0.1.0 → 0.1.1

package/src/devcouncil/council/prompts/arbiter.md

@@ -1,19 +1,19 @@
-Goal: {goal}
-
-Initial Requirements:
-{requirements_json}
-
-Plan A: {plan_a_json}
-Plan B: {plan_b_json}
-
-Critique of Plan B by Critic A: {critique_a_json}
-Critique of Plan A by Critic B: {critique_b_json}
-
-Rebuttal of Critic B by Planner A: {rebuttal_a_json}
-Rebuttal of Critic A by Planner B: {rebuttal_b_json}
-
-You are the arbiter engineering manager. Your goal is to produce the final, definitive set of requirements and tasks.
-- You do not decide by vibes.
-- High-severity unrefuted findings from critics must be incorporated into the final requirements or tasks.
-- If a planner successfully rebutted a finding, you may skip it.
-- Produce a single, coherent task graph.
+Goal: {goal}
+
+Initial Requirements:
+{requirements_json}
+
+Plan A: {plan_a_json}
+Plan B: {plan_b_json}
+
+Critique of Plan B by Critic A: {critique_a_json}
+Critique of Plan A by Critic B: {critique_b_json}
+
+Rebuttal of Critic B by Planner A: {rebuttal_a_json}
+Rebuttal of Critic A by Planner B: {rebuttal_b_json}
+
+You are the arbiter engineering manager. Your goal is to produce the final, definitive set of requirements and tasks.
+- You do not decide by vibes.
+- High-severity unrefuted findings from critics must be incorporated into the final requirements or tasks.
+- If a planner successfully rebutted a finding, you may skip it.
+- Produce a single, coherent task graph.

package/src/devcouncil/council/prompts/critic_a.md

@@ -1,10 +1,10 @@
-Requirements:
-{requirements_json}
-
-Target Plan:
-{target_plan_json}
-
-You are a hostile staff engineer reviewing another team's implementation plan.
-Find missing requirements, bad assumptions, missing tests, security risks, migration risks, and unverifiable claims.
-Do not praise. Do not rewrite the plan.
-Every finding must include a falsifiable_check.
+Requirements:
+{requirements_json}
+
+Target Plan:
+{target_plan_json}
+
+You are a hostile staff engineer reviewing another team's implementation plan.
+Find missing requirements, bad assumptions, missing tests, security risks, migration risks, and unverifiable claims.
+Do not praise. Do not rewrite the plan.
+Every finding must include a falsifiable_check.

package/src/devcouncil/council/prompts/critic_b.md

@@ -1,10 +1,10 @@
-Requirements:
-{requirements_json}
-
-Target Plan:
-{target_plan_json}
-
-You are a hostile staff engineer reviewing another team's implementation plan.
-Find missing requirements, bad assumptions, missing tests, security risks, migration risks, and unverifiable claims.
-Do not praise. Do not rewrite the plan.
-Every finding must include a falsifiable_check.
+Requirements:
+{requirements_json}
+
+Target Plan:
+{target_plan_json}
+
+You are a hostile staff engineer reviewing another team's implementation plan.
+Find missing requirements, bad assumptions, missing tests, security risks, migration risks, and unverifiable claims.
+Do not praise. Do not rewrite the plan.
+Every finding must include a falsifiable_check.

package/src/devcouncil/council/prompts/implementation_reviewer.md

@@ -1,16 +1,16 @@
-You are an expert software reviewer. Review the following code changes against the task requirements.
-Task: {task.title}
-Description: {task.description}
-
-Requirements:
-{requirements_json}
-
-Code Diff:
-{diff}
-
-Your task is to identify if the implementation is complete, correct, and follows best practices.
-- Identify missing edge cases.
-- Identify architectural drift.
-- Identify security risks not caught by static scans.
-
-Return a JSON object with 'is_satisfactory' and a list of 'findings' (as Gap objects).
+You are an expert software reviewer. Review the following code changes against the task requirements.
+Task: {task.title}
+Description: {task.description}
+
+Requirements:
+{requirements_json}
+
+Code Diff:
+{diff}
+
+Your task is to identify if the implementation is complete, correct, and follows best practices.
+- Identify missing edge cases.
+- Identify architectural drift.
+- Identify security risks not caught by static scans.
+
+Return a JSON object with 'is_satisfactory' and a list of 'findings' (as Gap objects).

package/src/devcouncil/council/prompts/planner_a.md

@@ -1,16 +1,16 @@
-Goal: {goal}
-
-Requirements:
-{requirements_json}
-
-Repository Map:
-{repo_map_json}
-
-Your task is to create a detailed implementation plan.
-- Break down the requirements into atomic implementation tasks.
-- For each task, specify which files will be created or modified.
-- Specify which tests are expected to verify the task.
-- Ensure each task maps back to at least one requirement.
-
-Role-specific instructions:
-You are the pragmatic tech lead. Optimize for simplicity and minimal dependencies.
+Goal: {goal}
+
+Requirements:
+{requirements_json}
+
+Repository Map:
+{repo_map_json}
+
+Your task is to create a detailed implementation plan.
+- Break down the requirements into atomic implementation tasks.
+- For each task, specify which files will be created or modified.
+- Specify which tests are expected to verify the task.
+- Ensure each task maps back to at least one requirement.
+
+Role-specific instructions:
+You are the pragmatic tech lead. Optimize for simplicity and minimal dependencies.

package/src/devcouncil/council/prompts/planner_b.md

@@ -1,16 +1,16 @@
-Goal: {goal}
-
-Requirements:
-{requirements_json}
-
-Repository Map:
-{repo_map_json}
-
-Your task is to create a detailed implementation plan.
-- Break down the requirements into atomic implementation tasks.
-- For each task, specify which files will be created or modified.
-- Specify which tests are expected to verify the task.
-- Ensure each task maps back to at least one requirement.
-
-Role-specific instructions:
-You are the production-readiness architect. Optimize for security, performance, edge cases, failure modes, and maintainability. Assume the first plan will miss subtle requirements.
+Goal: {goal}
+
+Requirements:
+{requirements_json}
+
+Repository Map:
+{repo_map_json}
+
+Your task is to create a detailed implementation plan.
+- Break down the requirements into atomic implementation tasks.
+- For each task, specify which files will be created or modified.
+- Specify which tests are expected to verify the task.
+- Ensure each task maps back to at least one requirement.
+
+Role-specific instructions:
+You are the production-readiness architect. Optimize for security, performance, edge cases, failure modes, and maintainability. Assume the first plan will miss subtle requirements.

package/src/devcouncil/council/prompts/rebuttal.md

@@ -1,10 +1,10 @@
-Original Plan:
-{original_plan_json}
-
-Critique Findings:
-{findings_json}
-
-You are the planner who created the original plan. Review the critique findings.
-- A finding can be rejected only with artifact evidence or strong justification.
-- A finding can be accepted and converted into a requirement/task/test.
-- No hand-wavy rebuttals.
+Original Plan:
+{original_plan_json}
+
+Critique Findings:
+{findings_json}
+
+You are the planner who created the original plan. Review the critique findings.
+- A finding can be rejected only with artifact evidence or strong justification.
+- A finding can be accepted and converted into a requirement/task/test.
+- No hand-wavy rebuttals.

package/src/devcouncil/council/prompts/spec_writer.md

@@ -1,12 +1,12 @@
-Goal: {goal}
-
-Repository Map:
-{repo_map_json}
-
-Your task is to draft the initial software specification for this goal.
-1. Identify functional and non-functional requirements.
-2. Extract any assumptions you are making about the codebase or architecture.
-3. List any blocking questions that the user must answer before implementation can proceed.
-
-Each requirement MUST have clear acceptance criteria with verification methods.
-Each assumption MUST have a confidence and impact level.
+Goal: {goal}
+
+Repository Map:
+{repo_map_json}
+
+Your task is to draft the initial software specification for this goal.
+1. Identify functional and non-functional requirements.
+2. Extract any assumptions you are making about the codebase or architecture.
+3. List any blocking questions that the user must answer before implementation can proceed.
+
+Each requirement MUST have clear acceptance criteria with verification methods.
+Each assumption MUST have a confidence and impact level.

package/src/devcouncil/domain/assumption.py

@@ -1,17 +1,17 @@
-from pydantic import BaseModel, Field
-from typing import Literal, List
-
-class Assumption(BaseModel):
-    id: str
-    statement: str
-    confidence: Literal["low", "medium", "high"]
-    impact: Literal["low", "medium", "high"]
-    reversible: bool
-    requires_user_confirmation: bool
-    linked_requirement_ids: List[str] = Field(default_factory=list)
-    status: Literal[
-        "open",
-        "confirmed",
-        "rejected",
-        "converted_to_requirement"
-    ] = "open"
+from pydantic import BaseModel, Field
+from typing import Literal, List
+
+class Assumption(BaseModel):
+    id: str
+    statement: str
+    confidence: Literal["low", "medium", "high"]
+    impact: Literal["low", "medium", "high"]
+    reversible: bool
+    requires_user_confirmation: bool
+    linked_requirement_ids: List[str] = Field(default_factory=list)
+    status: Literal[
+        "open",
+        "confirmed",
+        "rejected",
+        "converted_to_requirement"
+    ] = "open"

package/src/devcouncil/domain/critique.py

@@ -1,32 +1,32 @@
-from pydantic import BaseModel
-from typing import Literal, Optional
-
-class CritiqueFinding(BaseModel):
-    id: str
-    source_agent: str
-    target_plan_id: str
-    severity: Literal["low", "medium", "high", "critical"]
-    finding_type: Literal[
-        "missing_requirement",
-        "missing_task",
-        "missing_test",
-        "bad_assumption",
-        "architecture_risk",
-        "security_risk",
-        "performance_risk",
-        "dependency_risk",
-        "migration_risk",
-        "unverifiable_acceptance_criteria"
-    ]
-    claim: str
-    linked_requirement_id: Optional[str] = None
-    suggested_requirement: Optional[str] = None
-    suggested_task: Optional[str] = None
-    falsifiable_check: str
-    status: Literal[
-        "open",
-        "accepted",
-        "rejected",
-        "converted",
-        "needs_user"
-    ] = "open"
+from pydantic import BaseModel
+from typing import Literal, Optional
+
+class CritiqueFinding(BaseModel):
+    id: str
+    source_agent: str
+    target_plan_id: str
+    severity: Literal["low", "medium", "high", "critical"]
+    finding_type: Literal[
+        "missing_requirement",
+        "missing_task",
+        "missing_test",
+        "bad_assumption",
+        "architecture_risk",
+        "security_risk",
+        "performance_risk",
+        "dependency_risk",
+        "migration_risk",
+        "unverifiable_acceptance_criteria"
+    ]
+    claim: str
+    linked_requirement_id: Optional[str] = None
+    suggested_requirement: Optional[str] = None
+    suggested_task: Optional[str] = None
+    falsifiable_check: str
+    status: Literal[
+        "open",
+        "accepted",
+        "rejected",
+        "converted",
+        "needs_user"
+    ] = "open"

package/src/devcouncil/domain/evidence.py

@@ -1,27 +1,27 @@
-from pydantic import BaseModel
-from typing import Literal, List
-
-class CommandResult(BaseModel):
-    command: str
-    exit_code: int
-    stdout_path: str
-    stderr_path: str
-    summary: str
-
-class DiffEvidence(BaseModel):
-    task_id: str
-    changed_files: List[str]
-    added_files: List[str]
-    deleted_files: List[str]
-    diff_summary: str
-
-class VerificationEvidence(BaseModel):
-    __test__ = False  # Prevent pytest from collecting this as a test class
-    requirement_id: str
-    acceptance_criterion_id: str
-    command: str
-    status: Literal["passed", "failed", "not_run"]
-    evidence_summary: str
-
-# Backward-compatible alias
-TestEvidence = VerificationEvidence
+from pydantic import BaseModel
+from typing import Literal, List
+
+class CommandResult(BaseModel):
+    command: str
+    exit_code: int
+    stdout_path: str
+    stderr_path: str
+    summary: str
+
+class DiffEvidence(BaseModel):
+    task_id: str
+    changed_files: List[str]
+    added_files: List[str]
+    deleted_files: List[str]
+    diff_summary: str
+
+class VerificationEvidence(BaseModel):
+    __test__ = False  # Prevent pytest from collecting this as a test class
+    requirement_id: str
+    acceptance_criterion_id: str
+    command: str
+    status: Literal["passed", "failed", "not_run"]
+    evidence_summary: str
+
+# Backward-compatible alias
+TestEvidence = VerificationEvidence

package/src/devcouncil/domain/gap.py

@@ -1,26 +1,26 @@
-from pydantic import BaseModel, Field
-from typing import Literal, List, Optional
-
-class Gap(BaseModel):
-    id: str
-    severity: Literal["low", "medium", "high", "critical"]
-    gap_type: Literal[
-        "requirement_not_planned",
-        "task_not_implemented",
-        "planned_file_not_changed",
-        "orphan_diff",
-        "missing_test",
-        "test_failed",
-        "acceptance_criteria_unproven",
-        "assumption_violated",
-        "architecture_drift",
-        "security_risk",
-        "dependency_risk",
-        "migration_gap"
-    ]
-    requirement_id: Optional[str] = None
-    task_id: Optional[str] = None
-    description: str
-    evidence: List[str] = Field(default_factory=list)
-    recommended_fix: str
-    blocking: bool
+from pydantic import BaseModel, Field
+from typing import Literal, List, Optional
+
+class Gap(BaseModel):
+    id: str
+    severity: Literal["low", "medium", "high", "critical"]
+    gap_type: Literal[
+        "requirement_not_planned",
+        "task_not_implemented",
+        "planned_file_not_changed",
+        "orphan_diff",
+        "missing_test",
+        "test_failed",
+        "acceptance_criteria_unproven",
+        "assumption_violated",
+        "architecture_drift",
+        "security_risk",
+        "dependency_risk",
+        "migration_gap"
+    ]
+    requirement_id: Optional[str] = None
+    task_id: Optional[str] = None
+    description: str
+    evidence: List[str] = Field(default_factory=list)
+    recommended_fix: str
+    blocking: bool

package/src/devcouncil/domain/requirement.py

@@ -1,22 +1,22 @@
-from pydantic import BaseModel, Field
-from typing import Literal, List
-
-class AcceptanceCriterion(BaseModel):
-    id: str
-    description: str
-    verification_method: Literal[
-        "unit_test",
-        "integration_test",
-        "manual",
-        "static_check",
-        "llm_review"
-    ]
-    required: bool = True
-
-class Requirement(BaseModel):
-    id: str
-    title: str
-    description: str
-    priority: Literal["low", "medium", "high", "critical"]
-    source: Literal["user", "planner", "critic", "arbiter"]
-    acceptance_criteria: List[AcceptanceCriterion] = Field(default_factory=list)
+from pydantic import BaseModel, Field
+from typing import Literal, List
+
+class AcceptanceCriterion(BaseModel):
+    id: str
+    description: str
+    verification_method: Literal[
+        "unit_test",
+        "integration_test",
+        "manual",
+        "static_check",
+        "llm_review"
+    ]
+    required: bool = True
+
+class Requirement(BaseModel):
+    id: str
+    title: str
+    description: str
+    priority: Literal["low", "medium", "high", "critical"]
+    source: Literal["user", "planner", "critic", "arbiter"]
+    acceptance_criteria: List[AcceptanceCriterion] = Field(default_factory=list)

package/src/devcouncil/domain/task.py

@@ -1,26 +1,26 @@
-from pydantic import BaseModel, Field
-from typing import Literal, List
-
-class PlannedFile(BaseModel):
-    path: str
-    reason: str
-    allowed_change: Literal["create", "modify", "delete", "read_only"]
-
-class Task(BaseModel):
-    id: str
-    title: str
-    description: str
-    requirement_ids: List[str] = Field(default_factory=list)
-    acceptance_criterion_ids: List[str] = Field(default_factory=list)
-    planned_files: List[PlannedFile] = Field(default_factory=list)
-    expected_tests: List[str] = Field(default_factory=list)
-    allowed_commands: List[str] = Field(default_factory=list)
-    forbidden_changes: List[str] = Field(default_factory=list)
-    status: Literal[
-        "planned",
-        "ready",
-        "running",
-        "blocked",
-        "verified",
-        "done"
-    ] = "planned"
+from pydantic import BaseModel, Field
+from typing import Literal, List
+
+class PlannedFile(BaseModel):
+    path: str
+    reason: str
+    allowed_change: Literal["create", "modify", "delete", "read_only"]
+
+class Task(BaseModel):
+    id: str
+    title: str
+    description: str
+    requirement_ids: List[str] = Field(default_factory=list)
+    acceptance_criterion_ids: List[str] = Field(default_factory=list)
+    planned_files: List[PlannedFile] = Field(default_factory=list)
+    expected_tests: List[str] = Field(default_factory=list)
+    allowed_commands: List[str] = Field(default_factory=list)
+    forbidden_changes: List[str] = Field(default_factory=list)
+    status: Literal[
+        "planned",
+        "ready",
+        "running",
+        "blocked",
+        "verified",
+        "done"
+    ] = "planned"

package/src/devcouncil/execution/__init__.py

@@ -1 +1 @@
-
+

package/src/devcouncil/execution/context_builder.py

@@ -1,60 +1,60 @@
-from pathlib import Path
-from typing import List
-from devcouncil.domain.task import Task
-from devcouncil.domain.requirement import Requirement
-from devcouncil.utils.redaction import redact_string
-import json
-
-class ContextBuilder:
-    """Gathers repo-level and task-level context for agent prompts."""
-    
-    def __init__(self, project_root: Path):
-        self.project_root = project_root
-
-    def build_task_context(self, task: Task, requirements: List[Requirement]) -> str:
-        """Collects all relevant information for implementing a task."""
-        
-        # 1. Map requirements relevant to this task
-        req_map = {r.id: r for r in requirements}
-        task_reqs = [req_map[rid] for rid in task.requirement_ids if rid in req_map]
-        
-        # 2. Gather content of planned files (if they exist)
-        file_contents = {}
-        for pf in task.planned_files:
-            file_path = self.project_root / pf.path
-            if file_path.exists() and file_path.is_file():
-                try:
-                    raw = file_path.read_text(encoding="utf-8")
-                    file_contents[pf.path] = redact_string(raw)
-                except Exception:
-                    file_contents[pf.path] = "[Error reading file]"
-            else:
-                file_contents[pf.path] = "[New file - does not exist yet]"
-
-        # 3. Assemble the context string
-        context = {
-            "task": task.model_dump(),
-            "relevant_requirements": [r.model_dump() for r in task_reqs],
-            "file_contents": file_contents,
-            "project_structure": self.get_structure_summary(task)
-        }
-        
-        return json.dumps(context, indent=2)
-
-    def get_structure_summary(self, task: Task = None) -> List[str]:
-        """Simple list of files in the project for context."""
-        try:
+from pathlib import Path
+from typing import List
+from devcouncil.domain.task import Task
+from devcouncil.domain.requirement import Requirement
+from devcouncil.utils.redaction import redact_string
+import json
+
+class ContextBuilder:
+    """Gathers repo-level and task-level context for agent prompts."""
+    
+    def __init__(self, project_root: Path):
+        self.project_root = project_root
+
+    def build_task_context(self, task: Task, requirements: List[Requirement]) -> str:
+        """Collects all relevant information for implementing a task."""
+        
+        # 1. Map requirements relevant to this task
+        req_map = {r.id: r for r in requirements}
+        task_reqs = [req_map[rid] for rid in task.requirement_ids if rid in req_map]
+        
+        # 2. Gather content of planned files (if they exist)
+        file_contents = {}
+        for pf in task.planned_files:
+            file_path = self.project_root / pf.path
+            if file_path.exists() and file_path.is_file():
+                try:
+                    raw = file_path.read_text(encoding="utf-8")
+                    file_contents[pf.path] = redact_string(raw)
+                except Exception:
+                    file_contents[pf.path] = "[Error reading file]"
+            else:
+                file_contents[pf.path] = "[New file - does not exist yet]"
+
+        # 3. Assemble the context string
+        context = {
+            "task": task.model_dump(),
+            "relevant_requirements": [r.model_dump() for r in task_reqs],
+            "file_contents": file_contents,
+            "project_structure": self.get_structure_summary(task)
+        }
+        
+        return json.dumps(context, indent=2)
+
+    def get_structure_summary(self, task: Task = None) -> List[str]:
+        """Simple list of files in the project for context."""
+        try:
             import subprocess
             output = subprocess.check_output(
                 ["git", "ls-files", "--cached", "--others", "--exclude-standard"],
                 cwd=self.project_root,
                 stderr=subprocess.DEVNULL
             ).decode().splitlines()
-            
-            if task and task.planned_files:
-                planned_paths = {pf.path for pf in task.planned_files}
-                output = [p for p in output if p in planned_paths] + [p for p in output if p not in planned_paths]
-                
-            return output[:100] # Limit to avoid context overflow
-        except Exception:
-            return []
+            
+            if task and task.planned_files:
+                planned_paths = {pf.path for pf in task.planned_files}
+                output = [p for p in output if p in planned_paths] + [p for p in output if p not in planned_paths]
+                
+            return output[:100] # Limit to avoid context overflow
+        except Exception:
+            return []