ouroboros-ai 0.26.2.dev28__tar.gz → 0.26.2.dev32__tar.gz

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ouroboros-ai
-Version: 0.26.2.dev28
+Version: 0.26.2.dev32
 Summary: Specification-first workflow engine for AI coding agents. Works with Claude Code and Codex CLI.
 Author-email: Q00 <jqyu.lee@gmail.com>
 License-File: LICENSE

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/skills/brownfield/SKILL.md

@@ -49,7 +49,18 @@ This may take a moment...
 
 The scan response `text` already contains a pre-formatted numbered list with `[default]` markers. **Do NOT make any additional MCP calls to list or query repos.**
 
-**Display the repos in a compact 2-column grid** so the user can scan all repos quickly. Format the scan response `text` as two side-by-side columns. Include `*` markers for defaults exactly as they appear in the response.
+**Display the repos in a plain-text 2-column grid** (NOT a markdown table). Use a code block so columns align. Example:
+
+```
+Scan complete. 8 repositories registered.
+
+ 1. repo-alpha                   5. repo-epsilon
+ 2. repo-bravo *                 6. repo-foxtrot
+ 3. repo-charlie                 7. repo-golf *
+ 4. repo-delta                   8. repo-hotel
+```
+
+Include `*` markers for defaults exactly as they appear in the scan response.
 
 **If no repos found**, show:
 ```
@@ -59,7 +70,9 @@ Then stop.
 
 **Step 2: Default Selection**
 
-**IMMEDIATELY after showing the list**, use `AskUserQuestion` with the current default numbers from the scan response:
+**IMMEDIATELY after showing the list**, use `AskUserQuestion` with the current default numbers from the scan response.
+
+**If defaults exist**, show them as the recommended option:
 
 ```json
 {
@@ -75,7 +88,23 @@ Then stop.
 }
 ```
 
-The user can select the recommended defaults, choose "None", or type custom numbers.
+**If no defaults exist**, do NOT show a "(Recommended)" option — offer "None" and "Select repos" instead:
+
+```json
+{
+  "questions": [{
+    "question": "Which repos to set as default for interviews? Enter numbers like '6, 18, 19'.",
+    "header": "Default Repos",
+    "options": [
+      {"label": "None", "description": "No default repos — interviews will run in greenfield mode"},
+      {"label": "Select repos", "description": "Type repo numbers to set as default"}
+    ],
+    "multiSelect": false
+  }]
+}
+```
+
+The user can select the recommended defaults (if any), choose "None", or type custom numbers.
 
 After the user responds, use ONE MCP call to update all defaults at once:
 

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/skills/setup/SKILL.md

@@ -365,13 +365,43 @@ This may take a moment...
 
 The scan response `text` already contains a pre-formatted numbered list with `[default]` markers. **Do NOT make any additional MCP calls to list or query repos.**
 
-**Display the scan response text directly** — copy the full numbered list from the scan response `text` into your output as-is. Do not reformat, summarize, or truncate it. The user needs to see all repo numbers to pick defaults.
+**Display the repos in a plain-text 2-column grid** (NOT a markdown table). Use a code block so columns align. Example:
+
+```
+Scan complete. 8 repositories registered.
+
+ 1. repo-alpha                   5. repo-epsilon
+ 2. repo-bravo *                 6. repo-foxtrot
+ 3. repo-charlie                 7. repo-golf *
+ 4. repo-delta                   8. repo-hotel
+```
+
+Include `*` markers for defaults exactly as they appear in the scan response. Do not summarize or truncate the list. The user needs to see all repo numbers to pick defaults.
 
 **If no repos found**, skip the default selection prompt and proceed to Step 6.
 
 **Default repo selection — IMMEDIATELY after showing the list:**
 
-Use `AskUserQuestion` with the current default numbers from the scan response:
+Use `AskUserQuestion` with the current default numbers from the scan response.
+
+**If defaults exist**, show them as the recommended option:
+
+```json
+{
+  "questions": [{
+    "question": "Which repos to set as default for interviews? Enter numbers like '6, 18, 19'.",
+    "header": "Default Repos",
+    "options": [
+      {"label": "<current default numbers> (Recommended)", "description": "<current default names>"},
+      {"label": "None", "description": "Clear all defaults — interviews will run in greenfield mode"},
+      {"label": "Select repos", "description": "Type repo numbers to set as default"}
+    ],
+    "multiSelect": false
+  }]
+}
+```
+
+**If no defaults exist**, do NOT show a "(Recommended)" option — offer "None" and "Select repos" instead:
 
 ```json
 {
@@ -379,14 +409,15 @@ Use `AskUserQuestion` with the current default numbers from the scan response:
     "question": "Which repos to set as default for interviews? Enter numbers like '6, 18, 19'.",
     "header": "Default Repos",
     "options": [
-      {"label": "<current default numbers> (Recommended)", "description": "<current default names>"}
+      {"label": "None", "description": "No default repos — interviews will run in greenfield mode"},
+      {"label": "Select repos", "description": "Type repo numbers to set as default"}
     ],
     "multiSelect": false
   }]
 }
 ```
 
-Only ONE option: the current defaults as recommended. The user can "Type something" for custom numbers or "none".
+The user can select the recommended defaults (if any), choose "None", or type custom numbers.
 
 After the user responds, use ONE MCP call to update all defaults at once:
 

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/src/ouroboros/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.26.2.dev28'
-__version_tuple__ = version_tuple = (0, 26, 2, 'dev28')
+__version__ = version = '0.26.2.dev32'
+__version_tuple__ = version_tuple = (0, 26, 2, 'dev32')
 
 __commit_id__ = commit_id = None

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/src/ouroboros/evaluation/languages.py

@@ -127,6 +127,9 @@ LANGUAGE_PRESETS: dict[str, LanguagePreset] = {
     ),
 }
 
+_MAVEN_WRAPPER = "./mvnw"
+_MAVEN_WRAPPER_WINDOWS = "mvnw.cmd"
+
 # Ordered list of (marker_file, preset_key) for detection priority.
 # More specific markers come first (e.g. uv.lock before pyproject.toml).
 _DETECTION_RULES: list[tuple[str, str]] = [
@@ -173,6 +176,25 @@ def detect_language(working_dir: Path) -> LanguagePreset | None:
     return None
 
 
+def _resolve_maven_command(working_dir: Path) -> str:
+    """Resolve the safest Maven launcher for the current project/platform.
+
+    Prefers project-local wrapper scripts when they are runnable on the
+    current platform. Falls back to plain ``mvn`` when no suitable wrapper is
+    available.
+    """
+    if os.name == "nt":
+        wrapper = working_dir / _MAVEN_WRAPPER_WINDOWS
+        if wrapper.is_file():
+            return _MAVEN_WRAPPER_WINDOWS
+        return "mvn"
+
+    wrapper = working_dir / "mvnw"
+    if wrapper.is_file() and os.access(wrapper, os.X_OK):
+        return _MAVEN_WRAPPER
+    return "mvn"
+
+
 def _load_project_overrides(working_dir: Path) -> dict[str, Any] | None:
     """Load .ouroboros/mechanical.toml if it exists.
 
@@ -335,11 +357,18 @@ def build_mechanical_config(
     # Start with auto-detected preset
     preset = detect_language(working_dir)
 
+    build_command = preset.build_command if preset else None
+    test_command = preset.test_command if preset else None
+    if preset and preset.name == "java-maven":
+        maven_command = _resolve_maven_command(working_dir)
+        build_command = (maven_command, "clean", "compile")
+        test_command = (maven_command, "test")
+
     # Base command values from preset (or all None)
     current: dict[str, Any] = {
         "lint": preset.lint_command if preset else None,
-        "build": preset.build_command if preset else None,
-        "test": preset.test_command if preset else None,
+        "build": build_command,
+        "test": test_command,
         "static": preset.static_command if preset else None,
         "coverage": preset.coverage_command if preset else None,
         "timeout": 300,

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/src/ouroboros/persistence/brownfield.py

@@ -335,7 +335,7 @@ class BrownfieldStore:
             limit: Maximum number of rows to return. ``None`` means no limit.
 
         Returns:
-            List of BrownfieldRepo instances, ordered by name.
+            List of BrownfieldRepo instances, ordered by insertion order (rowid).
 
         Raises:
             PersistenceError: If the query fails.
@@ -345,7 +345,7 @@ class BrownfieldStore:
 
         try:
             async with engine.begin() as conn:
-                stmt = select(literal_column("rowid"), t).order_by(t.c.name)
+                stmt = select(literal_column("rowid"), t).order_by(literal_column("rowid"))
                 if offset > 0:
                     stmt = stmt.offset(offset)
                 if limit is not None:

{ouroboros_ai-0.26.2.dev28 → ouroboros_ai-0.26.2.dev32}/tests/unit/evaluation/test_languages.py

@@ -54,6 +54,17 @@ class TestDetectLanguage:
         preset = detect_language(tmp_path)
         assert preset is not None
         assert preset.name == "java-maven"
+        assert preset.build_command == ("mvn", "clean", "compile")
+        assert preset.test_command == ("mvn", "test")
+
+    def test_detect_java_maven_wrapper_does_not_change_preset(self, tmp_path: Path) -> None:
+        (tmp_path / "pom.xml").touch()
+        (tmp_path / "mvnw").touch()
+        preset = detect_language(tmp_path)
+        assert preset is not None
+        assert preset.name == "java-maven"
+        assert preset.build_command == ("mvn", "clean", "compile")
+        assert preset.test_command == ("mvn", "test")
 
     def test_detect_node_npm(self, tmp_path: Path) -> None:
         (tmp_path / "package.json").touch()
@@ -162,6 +173,12 @@ class TestParseCommand:
     def test_allowed_executable(self) -> None:
         assert _parse_command("cargo test") == ("cargo", "test")
 
+    def test_path_based_maven_wrapper_override_is_blocked(self) -> None:
+        assert _parse_command("./mvnw test") is None
+
+    def test_path_traversal_maven_wrapper_override_is_blocked(self) -> None:
+        assert _parse_command("../../tmp/mvnw test") is None
+
 
 class TestBuildMechanicalConfig:
     """Tests for build_mechanical_config()."""
@@ -252,6 +269,55 @@ class TestBuildMechanicalConfig:
         assert config.coverage_command is None
         assert config.working_dir == tmp_path
 
+    def test_auto_detect_java_maven_prefers_executable_wrapper(self, tmp_path: Path) -> None:
+        (tmp_path / "pom.xml").touch()
+        wrapper = tmp_path / "mvnw"
+        wrapper.touch()
+        wrapper.chmod(0o755)
+        config = build_mechanical_config(tmp_path)
+        assert config.build_command == ("./mvnw", "clean", "compile")
+        assert config.test_command == ("./mvnw", "test")
+
+    def test_auto_detect_java_maven_falls_back_when_wrapper_not_executable(
+        self, tmp_path: Path
+    ) -> None:
+        (tmp_path / "pom.xml").touch()
+        wrapper = tmp_path / "mvnw"
+        wrapper.touch()
+        wrapper.chmod(0o644)
+        config = build_mechanical_config(tmp_path)
+        assert config.build_command == ("mvn", "clean", "compile")
+        assert config.test_command == ("mvn", "test")
+
+    def test_auto_detect_java_maven_uses_windows_wrapper(self, tmp_path: Path, monkeypatch) -> None:
+        (tmp_path / "pom.xml").touch()
+        (tmp_path / "mvnw.cmd").touch()
+        monkeypatch.setattr("ouroboros.evaluation.languages.os.name", "nt")
+        config = build_mechanical_config(tmp_path)
+        assert config.build_command == ("mvnw.cmd", "clean", "compile")
+        assert config.test_command == ("mvnw.cmd", "test")
+
+    def test_auto_detect_java_maven_falls_back_when_wrapper_is_directory(
+        self, tmp_path: Path
+    ) -> None:
+        (tmp_path / "pom.xml").touch()
+        wrapper = tmp_path / "mvnw"
+        wrapper.mkdir()
+        config = build_mechanical_config(tmp_path)
+        assert config.build_command == ("mvn", "clean", "compile")
+        assert config.test_command == ("mvn", "test")
+
+    def test_auto_detect_java_maven_falls_back_when_windows_wrapper_is_directory(
+        self, tmp_path: Path, monkeypatch
+    ) -> None:
+        (tmp_path / "pom.xml").touch()
+        wrapper = tmp_path / "mvnw.cmd"
+        wrapper.mkdir()
+        monkeypatch.setattr("ouroboros.evaluation.languages.os.name", "nt")
+        config = build_mechanical_config(tmp_path)
+        assert config.build_command == ("mvn", "clean", "compile")
+        assert config.test_command == ("mvn", "test")
+
     def test_no_toml_file_no_error(self, tmp_path: Path) -> None:
         """Missing .ouroboros/mechanical.toml is not an error."""
         (tmp_path / "build.zig").touch()