fix(cli): fix integration test and add changelog for actor context list regex (#6500 )

- Fix Robot Framework integration test Actor Context List Supports Regex Filter to use --context-dir with named actor context subdirectories instead of project context files, resolving the CI integration_tests failure - Add CHANGELOG.md entry for the [REGEX] positional argument feature - Rebase on master to resolve merge conflicts ISSUES CLOSED: #6500
2026-05-09 02:43:11 +00:00 · 2026-05-05 19:56:00 +00:00 · 2026-05-05 19:26:02 +00:00 · 2026-05-05 17:48:39 +00:00 · 2026-05-05 17:48:39 +00:00 · 2026-05-05 17:48:37 +00:00
31 changed files with 1826 additions and 101 deletions
@@ -245,6 +245,16 @@ each work group's fetch algorithm:
 The prompt body to pass to workers you spawn:
  ```
    Implement or fix the indicated issue or pull request.
+
+    PR Compliance Checklist (MANDATORY — complete ALL items before creating a PR):
+    [ ] 1. CHANGELOG.md — add entry under [Unreleased] section
+    [ ] 2. CONTRIBUTORS.md — add or update contribution entry
+    [ ] 3. Commit footer — include `ISSUES CLOSED: #<issue-number>` in the commit message
+    [ ] 4. CI passes — all quality gates and tests green before requesting review
+    [ ] 5. BDD/Behave tests — added or updated for the changed behaviour
+    [ ] 6. Epic reference — PR description references the parent Epic issue number
+    [ ] 7. Labels — applied via forgejo-label-manager: State/In Review, Priority/<level>, MoSCoW/<level>, Type/<type>
+    [ ] 8. Milestone — PR assigned to the earliest open milestone matching the issue
  ```
 ```

@@ -5,6 +5,15 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).

 ## [Unreleased]

+### Changed
+
+- **`agents session list` now displays full 26-character session ULIDs** (#10970): The Rich table
+  and Summary panel ("Most Recent" / "Oldest") previously showed only the first 8 characters of
+  each session ULID. This made the output unusable for copy-paste into `session tell`,
+  `session show`, `session delete`, and `session export`, all of which require the full
+  26-character identifier. The full ULID is now displayed in all output formats (Rich, plain,
+  JSON, YAML, table).
+
 ### Security

 - **aiohttp upgraded to >=3.13.4 to remediate CVE-2026-34513 and CVE-2026-34515** (#1549, #1544):
@@ -15,6 +24,16 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
  versions (<3.13.4) cannot be installed even if upstream transitive dependencies have loose
  version constraints.

+### Fixed
+
+- **Implementation Supervisor PR Compliance Checklist** (#9824): Added a mandatory
+  8-item PR Compliance Checklist to the worker prompt body in `implementation-supervisor.md`
+  that every implementation worker must complete before creating a PR. Checklist covers:
+  CHANGELOG.md update, CONTRIBUTORS.md update, commit footer (`ISSUES CLOSED: #N`),
+  CI verification, BDD tests, Epic reference, label application via `forgejo-label-manager`,
+  and milestone assignment. This eliminates systemic PR merge blockers caused by workers
+  omitting required items.
+
 ### Changed

 - Restored `benchmark-regression` CI job to `master.yml` with `pull_request` trigger guard
@@ -50,7 +69,6 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
 - `agents actor context clear` command to reset actor message history and
  state while preserving the underlying context directory via `ContextManager`
  (#6370).
- **Quick Start Guide** (PR #9245): Added `docs/quickstart.md` with an end-to-end quick start guide covering prerequisites, installation, project creation, resource registration, plan/apply workflow, and troubleshooting. Updated `mkdocs.yml` navigation to include the Quick Start page.

 - **Plan checkpoint management CLI commands** (#8683): Added `agents plan checkpoint-list <plan-id>` and `agents plan checkpoint-delete <checkpoint-id>` commands. Listing output now highlights checkpoint ID, type, created timestamp, reason, phase, and decision linkage with a concise field summary footer across rich/table/json/yaml formats. Deletion supports batch IDs, interactive confirmation (skip with `--yes`), and structured JSON/YAML responses for automation-friendly scripting.
 - **Invariant Remove CLI Command** (#8530): Implemented `agents invariant remove <id>` command that soft-deletes an invariant by ID. The command displays a confirmation prompt before removal (bypassable with `--yes`/`-y`), outputs the removed invariant ID on success, and shows a clear error message when the invariant ID does not exist. Supports `--format` flag for JSON and YAML output. Full BDD test coverage and Robot Framework integration tests included.
@@ -79,6 +97,19 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).

 ### Fixed

+- **`LLMTraceRepository.save()` premature commit breaks UnitOfWork transactions** (#7505):
+  Replaced the unconditional `session.commit()` in `LLMTraceRepository.save()` with a
+  dual-path implementation that respects the UnitOfWork (UoW) pattern. When an external
+  session is provided (UoW mode), the method now calls only `session.flush()`, leaving
+  transaction control to the caller. When no session is provided (standalone mode), the
+  method creates its own session, flushes, commits, and closes it to ensure durable
+  persistence. This eliminates three data-integrity violations: premature commit of outer
+  UoW transactions, loss of rollback capability for subsequent failures, and a mismatch
+  between the class docstring ("Callers are responsible for commit") and the implementation.
+  Input validation for the `trace` argument was also added. Two new BDD scenarios verify
+  the session contract: `Repository save() calls flush not commit` and `LLM trace rolled
+  back when UnitOfWork transaction rolls back`.
+
 - **git_tools._get_base_env() TOCTOU Race Condition** (#7619): Fixed a
  Time-Of-Check-To-Time-Of-Use race condition in `git_tools._get_base_env()`
  where two concurrent threads could both observe `_BASE_ENV is None`, both
@@ -265,6 +296,14 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).

 ### Added

+- **ACMS Index Data Model and File Traversal Engine** (#9579): Implements the
+  foundational ACMS index data model with structured fields for file metadata
+  (path, size, last modified, type), tag system, and hot/warm/cold/archive
+  storage tier assignment. Introduces a timeout-safe large-project file traversal
+  engine capable of handling 10,000+ files without memory exhaustion through
+  chunked processing. Provides a complete index entry pipeline for creation,
+  storage, and retrieval with full queryability by path, tag, type, and recency.
+
 - **ACMS Large-Project Indexing BDD Coverage** (#8726): Added 7 Behave scenarios
  covering walk-based indexing of 10,000+ files without timeout, binary-file
  skipping, oversized-file skipping, git-checkout indexing, fallback to walk when
@@ -273,6 +312,7 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
  syscalls). Added `timeout=120` to git subprocess calls to prevent CI hangs.
  Cached `get_scoped_view` results in `When` steps to avoid redundant re-queries
  in `Then` steps.
+
 - **Agent Evolution Pool Supervisor PR Metadata Assignment** (#7888): The
  agent-evolution-pool-supervisor now automatically looks up the Type/Automation
  label and the earliest open milestone from the repository before dispatching
@@ -303,6 +343,12 @@ The format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/).
  had `@tdd_expected_fail` removed and now run as permanent regression guards.
  Net result: 629 features active in CI (up from ~545), zero `@skip` tags remain.

+- **Actor Context List Regex Filter** (#6500): `agents actor context list` now
+  accepts an optional `[REGEX]` positional argument to filter listed named
+  contexts by a regular expression pattern. Invalid patterns are reported with
+  a clear error message. The deprecated `agents context list` alias receives
+  the same filtering capability.
+
 - **Git Worktree Sandbox Apply** (#4454): The `plan apply` command now merges
  LLM-generated changes via `git merge` from an isolated worktree branch
  instead of flat `shutil.copy2`.  Displays spec-aligned Apply Summary
@@ -7,7 +7,6 @@
 * Jeffrey Phillips Freeman <jeffrey.freeman@syncleus.com>
 * Luis Mendes <luis.p.mendes@gmail.com>
 * Rui Hu <rui.hu@cleverthis.com>
-* HAL 9000 <hal9000@cleverthis.com>

 # Details

@@ -25,7 +24,11 @@ Below are some of the specific details of various contributions.
 * This project was made possible thanks to considerable donation of time, money, and resources by CleverThis, Inc.
 * HAL 9000 has contributed automated bug fixes, CLI output formatting improvements, and ongoing maintenance as part of the CleverAgents automation system.
 * HAL 9000 has contributed the file edit encoding parameter fix (PR #8258 / issue #7559).
-* HAL 9000 has contributed the architecture-pool-supervisor milestone assignment feature (PR #8188 / issue #7521): added `forgejo_update_pull_request` permission and documented the PR workflow for major spec changes, enabling automatic milestone assignment for specification PRs.
+<<* HAL 9000 has contributed the architecture-pool-supervisor milestone assignment feature (PR #8188 / issue #7521): added `forgejo_update_pull_request` permission and documented the PR workflow for major spec changes, enabling automatic milestone assignment for specification PRs.
 * HAL 9000 has contributed the git worktree TOCTOU race condition fix (PR #8178 / issue #7507): replaced the unsafe mkdtemp() + rmdir() pattern with a parent-directory approach to eliminate the race window in concurrent git worktree operations.
 * HAL 9000 has contributed the git_tools TOCTOU race condition fix (PR #8255 / issue #7619): eliminated the Time-Of-Check-To-Time-Of-Use race in `_get_base_env()` by adding double-checked locking with a module-level `threading.Lock`, preventing concurrent threads from writing conflicting environment snapshots.
+* HAL 9000 has contributed the mandatory PR compliance checklist to `implementation-supervisor.md` (#9824): added an 8-item checklist to the worker prompt body with concrete items covering CHANGELOG.md, CONTRIBUTORS.md, commit footer, CI verification, BDD tests, Epic reference, labels, and milestone assignment to eliminate systemic PR merge blockers.
 * HAL 9000 has contributed comprehensive milestone documentation for v3.6.0 (Advanced Concepts & Deferred Features) and v3.7.0 (TUI Implementation) (PR #9903): split into sub-documents covering context strategies, LLM backends, resource types, A2A rename, container tool execution, scope chain resolution, cost/safety budgets, E2E workflow tests, code review examples, plugin architecture, TUI layout, persona system, reference/command input, session management, configuration, and TuiMaterializer integration.
+* HAL 9000 has contributed the LLMTraceRepository data-integrity fix (PR #8185 / issue #7505): replaced the unconditional `session.commit()` in `LLMTraceRepository.save()` with a dual-path implementation that respects the UnitOfWork pattern — flushing only when an external session is provided, and flushing + committing + closing when operating standalone. This eliminates premature transaction commits, loss of rollback capability, and a docstring/implementation mismatch.
+* HAL 9000 has contributed the ACMS Index Data Model and File Traversal Engine (PR #9664 / issue #9579): foundational data structures for indexed context entries with hot/warm/cold/archive storage tier classification, tag system, and a timeout-safe chunked file traversal engine for large projects with 10,000+ files.
+* HAL 9000 has contributed the actor context list regex filter feature (PR #6741 / issue #6500): added an optional `[REGEX]` positional argument to `agents actor context list` command to filter named contexts by a regular expression pattern, including BDD test coverage and Robot Framework integration test with `--context-dir` support for isolation.
@@ -1,45 +0,0 @@
-# Quick Start Guide
-
-This quick start guide will walk you through creating a new project, registering a resource, running a plan, and applying changes with CleverAgents.
-
-## Prerequisites
-
- Python 3.11+ and virtualenv
- Git
- A working CleverAgents installation (see development/testing.md for details)
-
-## Install (local development)
-
-```bash
-python -m venv .venv
-source .venv/bin/activate
-pip install -e .[dev]
-```
-
-## Create a new project
-
-```bash
-# Create a new directory for your project
-mkdir my-project && cd my-project
-# Initialize a CleverAgents project (example command)
-cleveragents init --name my-project
-```
-
-## Register a resource
-
-Create a resource file under `resources/` (example YAML) and register it with the CLI or API.
-
-## Plan and apply
-
-```bash
-# Create a plan using an action on your project
-cleveragents plan use <action-name> --project my-project
-# List plans to find the plan ID
-cleveragents plan list
-# Review the plan, then apply it by plan ID
-cleveragents plan apply <plan-id>
-```
-
-## Troubleshooting
-
-If you encounter issues running the examples above, consult `docs/development/testing.md` and the project README for local development tips.
@@ -72,8 +72,8 @@ The `rich` format renders a sessions table with columns: **ID**, **Name**, **Act
 | Field | Description |
 |-------|-------------|
 | Total | Number of sessions |
-| Most Recent | Name or truncated ID of the most recently updated session |
-| Oldest | Name or truncated ID of the oldest session |
+| Most Recent | Name or full ULID of the most recently updated session |
+| Oldest | Name or full ULID of the oldest session |
 | Total Messages | Sum of messages across all sessions |
 | Storage | Estimated storage used |

@@ -85,7 +85,7 @@ Followed by a `✓ OK N sessions listed` success message.
 {
  "sessions": [
    {
-      "id": "01HXYZ...",
+      "id": "01HXYZ4M1Q3F0R0E5HR8K5T8A",
      "name": "my-session",
      "actor": "openai/gpt-4",
      "messages": 5,
@@ -296,13 +296,13 @@ $ python -m cleveragents session list
 ┏━━━━━━━━━━┳━━━━━━━━━━━┳━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┓
 ┃ ID       ┃ Name      ┃ Actor  ┃ Messages ┃ Updated          ┃
 ┡━━━━━━━━━━╇━━━━━━━━━━━╇━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━┩
-│ 01KNKK4Q │ (unnamed) │ (none) │        0 │ 2026-04-07 09:07 │
+│ 01KNKK4Q9GZ0TRR5B0NEJYGMWH │ (unnamed) │ (none) │        0 │ 2026-04-07 09:07 │
 └──────────┴───────────┴────────┴──────────┴──────────────────┘

 ╭────────────────────────────────── Summary ───────────────────────────────────╮
 │ Total:          1                                                            │
-│ Most Recent:    01KNKK4Q                                                     │
-│ Oldest:         01KNKK4Q                                                     │
+│ Most Recent:    01KNKK4Q9GZ0TRR5B0NEJYGMWH                                                     │
+│ Oldest:         01KNKK4Q9GZ0TRR5B0NEJYGMWH                                                     │
 │ Total Messages: 0                                                            │
 │ Storage:        0 KB                                                         │
 ╰──────────────────────────────────────────────────────────────────────────────╯
@@ -311,7 +311,7 @@ $ python -m cleveragents session list
 ```

 **What's Happening:**
-The session list shows all sessions with their truncated ID, optional name, bound actor, message count, and last update time. The summary panel provides aggregate statistics across all sessions.
+The session list shows all sessions with their full ULID, optional name, bound actor, message count, and last update time. The summary panel provides aggregate statistics across all sessions.

 ---

@@ -339,8 +339,8 @@ $ python -m cleveragents session list --format json
    ],
    "summary": {
      "total": 1,
-      "most_recent": "01KNKK4Q",
-      "oldest": "01KNKK4Q",
+      "most_recent": "01KNKK4Q9GZ0TRR5B0NEJYGMWH",
+      "oldest": "01KNKK4Q9GZ0TRR5B0NEJYGMWH",
      "total_messages": 0,
      "storage": "0 KB"
    }
@@ -469,7 +469,7 @@ $ python -m cleveragents session list
 ┏━━━━━━━━━━┳━━━━━━━━━━━┳━━━━━━━━┳━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┓
 ┃ ID       ┃ Name      ┃ Actor  ┃ Messages ┃ Updated          ┃
 ┡━━━━━━━━━━╇━━━━━━━━━━━╇━━━━━━━━╇━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━┩
-│ 01KNKK4Q │ (unnamed) │ (none) │        0 │ 2026-04-07 09:07 │
+│ 01KNKK4Q9GZ0TRR5B0NEJYGMWH │ (unnamed) │ (none) │        0 │ 2026-04-07 09:07 │
 └──────────┴───────────┴────────┴──────────┴──────────────────┘
 ✓ OK 1 sessions listed
 ```
@@ -180,14 +180,14 @@ $ python -m cleveragents session list
 ┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
 ┃ ID       ┃ Name       ┃ Actor          ┃ Messages ┃ Updated          ┃
 ┡━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
-│ 01HXYZ4M │ (unnamed)  │ openai/gpt-4o  │        3 │ 2026-04-07 09:22 │
-│ 01HXYZ3K │ (unnamed)  │ (none)         │        0 │ 2026-04-07 09:15 │
+│ 01HXYZ4M1Q3F0R0E5HR8K5T8A │ (unnamed)  │ openai/gpt-4o  │        3 │ 2026-04-07 09:22 │
+│ 01HXYZ3K9P2E9Q9D4GQ7J4S7Z │ (unnamed)  │ (none)         │        0 │ 2026-04-07 09:15 │
 └──────────┴────────────┴────────────────┴──────────┴──────────────────┘

 ╭──────────────────────────────────────────── Summary ─────────────────────────────────────────────╮
 │ Total:          2                                                                                 │
-│ Most Recent:    01HXYZ4M                                                                          │
-│ Oldest:         01HXYZ3K                                                                          │
+│ Most Recent:    01HXYZ4M1Q3F0R0E5HR8K5T8A                                                                          │
+│ Oldest:         01HXYZ3K9P2E9Q9D4GQ7J4S7Z                                                                          │
 │ Total Messages: 3                                                                                 │
 │ Storage:        0 KB                                                                              │
 ╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
@@ -196,8 +196,8 @@ $ python -m cleveragents session list
 ```

 **What's Happening:**
-The list command renders a Rich table with five columns: truncated ID (first
-8 characters for readability), optional name, bound actor, message count, and
+The list command renders a Rich table with five columns: the full session
+ULID (26 characters), optional name, bound actor, message count, and
 last update time. The **Summary** panel below shows aggregate statistics
 including total sessions, most recent, oldest, total message count, and
 storage used.
@@ -229,8 +229,8 @@ $ python -m cleveragents session list --format json
  ],
  "summary": {
    "total": 2,
-    "most_recent": "01HXYZ4M",
-    "oldest": "01HXYZ3K",
+    "most_recent": "01HXYZ4M1Q3F0R0E5HR8K5T8A",
+    "oldest": "01HXYZ3K9P2E9Q9D4GQ7J4S7Z",
    "total_messages": 3,
    "storage": "0 KB"
  }
@@ -773,7 +773,7 @@ $ python -m cleveragents session list
 ✓ OK 2 sessions listed

 $ python -m cleveragents session list --format json
-{"sessions": [...], "summary": {"total": 2, "most_recent": "01HXYZ4M", ...}}
+{"sessions": [...], "summary": {"total": 2, "most_recent": "01HXYZ4M1Q3F0R0E5HR8K5T8A", ...}}

 $ python -m cleveragents session tell --session 01HXYZ4M1Q3F0R0E5HR8K5T8A "What is the capital of France?"
 user: What is the capital of France?
@@ -1723,8 +1723,8 @@ None.
    ╭─ Sessions ───────────────────────────────────────────────────────────────────╮
    │ <span style="color: cyan; font-weight: 600;">ID</span>        <span style="color: cyan; font-weight: 600;">Name</span>             <span style="color: cyan; font-weight: 600;">Actor</span>               <span style="color: cyan; font-weight: 600;">Messages</span>  <span style="color: cyan; font-weight: 600;">Updated</span>             │
    │ <span style="opacity: 0.7;">────────  ───────────────  ──────────────────  ────────  ────────────────</span>    │
-    │ 01HXM2A6  weekly-planning  local/orchestrator  6         2026-02-08 12:44    │
-    │ 01HXM1F2  refactor-sprint  local/orchestrator  14        2026-02-07 18:11    │
+    │ 01HXM2A61MQHZ4MRBAY3MPNJTN  weekly-planning  local/orchestrator  6         2026-02-08 12:44    │
+    │ 01HXM1F21MQHZ4MRBAY3MPNJTN  refactor-sprint  local/orchestrator  14        2026-02-07 18:11    │
    ╰──────────────────────────────────────────────────────────────────────────────╯

    ╭─ Summary ────────────────────╮
@@ -1746,8 +1746,8 @@ None.
    Sessions
    ID        Name             Actor               Messages  Updated
    --------  ---------------  ------------------  --------  ----------------
-    01HXM2A6  weekly-planning  local/orchestrator  6         2026-02-08 12:44
-    01HXM1F2  refactor-sprint  local/orchestrator  14        2026-02-07 18:11
+    01HXM2A61MQHZ4MRBAY3MPNJTN  weekly-planning  local/orchestrator  6         2026-02-08 12:44
+    01HXM1F21MQHZ4MRBAY3MPNJTN  refactor-sprint  local/orchestrator  14        2026-02-07 18:11

    Summary
      Total: 2
@@ -1769,14 +1769,14 @@ None.
      "data": {
        "sessions": [
          {
-            "id": "01HXM2A6",
+            "id": "01HXM2A61MQHZ4MRBAY3MPNJTN",
            "name": "weekly-planning",
            "actor": "local/orchestrator",
            "messages": 6,
            "updated": "2026-02-08T12:44:00Z"
          },
          {
-            "id": "01HXM1F2",
+            "id": "01HXM1F21MQHZ4MRBAY3MPNJTN",
            "name": "refactor-sprint",
            "actor": "local/orchestrator",
            "messages": 14,
@@ -1804,12 +1804,12 @@ None.
    exit_code: 0
    data:
      sessions:
-        - id: 01HXM2A6
+        - id: 01HXM2A61MQHZ4MRBAY3MPNJTN
          name: weekly-planning
          actor: local/orchestrator
          messages: 6
          updated: "2026-02-08T12:44:00Z"
-        - id: 01HXM1F2
+        - id: 01HXM1F21MQHZ4MRBAY3MPNJTN
          name: refactor-sprint
          actor: local/orchestrator
          messages: 14
@@ -0,0 +1,138 @@
+Feature: ACMS Index Data Model and File Traversal Engine
+  As a developer
+  I want to index large projects with 10,000+ files
+  So that I can efficiently query and manage context entries at scale
+
+  Background:
+    Given I have an ACMS index
+    And I have a file traversal engine with chunk size 100
+
+  Scenario: Create an index entry with file metadata
+    When I create an index entry with:
+      | path | /project/src/main.py |
+      | file_type | python |
+      | size_bytes | 1024 |
+    Then the index entry should have path "/project/src/main.py"
+    And the index entry should have file type "python"
+    And the index entry should have size 1024 bytes
+
+  Scenario: Add tags to an index entry
+    Given I have an index entry with path "/project/src/main.py"
+    When I add tag "core" to the entry
+    And I add tag "important" to the entry
+    Then the entry should have tag "core"
+    And the entry should have tag "important"
+    And the entry should have 2 tags
+
+  Scenario: Set tier level for an index entry
+    Given I have an index entry with path "/project/src/main.py"
+    When I set the tier level to "hot"
+    Then the entry should have tier level "hot"
+
+  Scenario: Add entry to index
+    Given I have an index entry with path "/project/src/main.py"
+    When I add the entry to the index
+    Then the index should contain 1 entry
+    And I should be able to retrieve the entry by path "/project/src/main.py"
+
+  Scenario: Query index by path pattern
+    Given I have an index with entries:
+      | path | file_type |
+      | /project/src/main.py | python |
+      | /project/src/utils.py | python |
+      | /project/tests/test_main.py | python |
+      | /project/docs/readme.md | markdown |
+    When I query the index by path pattern "src"
+    Then I should get 2 results
+    And the results should include "/project/src/main.py"
+    And the results should include "/project/src/utils.py"
+
+  Scenario: Query index by file type
+    Given I have an index with entries:
+      | path | file_type |
+      | /project/src/main.py | python |
+      | /project/src/utils.py | python |
+      | /project/src/app.js | javascript |
+      | /project/docs/readme.md | markdown |
+    When I query the index by file type "python"
+    Then I should get 2 results
+    And all results should have file type "python"
+
+  Scenario: Query index by tag
+    Given I have an index with entries:
+      | path | tags |
+      | /project/src/main.py | core,important |
+      | /project/src/utils.py | supporting |
+      | /project/tests/test_main.py | test,important |
+    When I query the index by tag "important"
+    Then I should get 2 results
+    And the results should include "/project/src/main.py"
+    And the results should include "/project/tests/test_main.py"
+
+  Scenario: Query index by tier level
+    Given I have an index with entries:
+      | path | tier |
+      | /project/src/main.py | hot |
+      | /project/src/utils.py | warm |
+      | /project/tests/test_main.py | cold |
+    When I query the index by tier level "hot"
+    Then I should get 1 result
+    And the result should have path "/project/src/main.py"
+
+  Scenario: Query index by recency
+    Given I have an index with entries from different dates
+    When I query the index for entries modified after "2026-04-01"
+    Then I should get entries modified after that date
+
+  Scenario: Traverse and index a directory with multiple files
+    Given I have a test directory with 50 files
+    When I traverse and index the directory
+    Then the index should contain 50 entries
+    And all entries should have valid file paths
+
+  Scenario: Handle large project traversal with chunked processing
+    Given I have a test directory with 1000 files
+    When I traverse and index the directory with chunk size 100
+    Then the index should contain 1000 entries
+    And the traversal should complete without timeout
+
+  Scenario: Exclude patterns during traversal
+    Given I have a test directory with files including:
+      | path |
+      | /project/src/main.py |
+      | /project/.git/config |
+      | /project/__pycache__/main.cpython-39.pyc |
+      | /project/src/utils.py |
+    When I traverse and index the directory excluding ".git" and "__pycache__"
+    Then the index should contain 2 entries
+    And the index should not contain ".git" paths
+    And the index should not contain "__pycache__" paths
+
+  Scenario: Get all entries from index
+    Given I have an index with 5 entries
+    When I get all entries from the index
+    Then I should get 5 results
+
+  Scenario: Get entry count from index
+    Given I have an index with 10 entries
+    When I get the entry count
+    Then the count should be 10
+
+  Scenario: Remove entry from index
+    Given I have an index with 3 entries
+    When I remove an entry by path
+    Then the index should contain 2 entries
+
+  Scenario: Combined query with multiple filters
+    Given I have an index with entries:
+      | path | file_type | tags | tier |
+      | /project/src/main.py | python | core,important | hot |
+      | /project/src/utils.py | python | supporting | warm |
+      | /project/tests/test_main.py | python | test | cold |
+      | /project/docs/readme.md | markdown | docs | cold |
+    When I query the index with filters:
+      | path_pattern | src |
+      | file_type | python |
+      | tier | hot |
+    Then I should get 1 result
+    And the result should have path "/project/src/main.py"
@@ -6,6 +6,15 @@ Feature: Actor context clear, remove, export, and import commands
  Background:
    Given a temporary context directory for actor context tests

+  @actor_context_list_regex
+  Scenario: List actor contexts filtered by regex
+    Given an actor context named "docs" exists
+    And an actor context named "notes" exists
+    When I run actor context list "docs"
+    Then the actor context list command should succeed
+    And the list output should contain "docs"
+    And the list output should not contain "notes"
+
  # ── context clear ─────────────────────────────────────────

  Scenario: Clear a named actor context
@@ -1714,6 +1714,12 @@ Feature: Consolidated Misc
    And the temporary connection should be closed afterward


+  Scenario: get_current_revision uses check_same_thread=False for SQLite engines
+    Given a migration runner configured for "sqlite:///:memory:"
+    When I request the current revision from the database
+    Then the SQLite engine for get_current_revision should use check_same_thread=False
+
+
  Scenario: File-based SQLite database directory is created if missing
    Given a migration runner configured for "sqlite:///tmp/test-db/mydb.db"
    When I initialize or upgrade a file-based SQLite database
@@ -686,6 +686,12 @@ def after_scenario(context, scenario):
            pass  # Ignore cleanup errors
        context.test_dir = None

+    # Clean up TemporaryDirectory objects created by ACMS index traversal tests
+    if hasattr(context, "temp_dir") and context.temp_dir is not None:
+        with contextlib.suppress(Exception):
+            context.temp_dir.cleanup()
+        context.temp_dir = None
+
    # Clean up environment variables set during tests
    if hasattr(context, "env_vars_to_clean"):
        for key in context.env_vars_to_clean:
@@ -0,0 +1,58 @@
+@mock_only
+Feature: PR Compliance Checklist in Implementation Supervisor
+
+  As an implementation supervisor
+  I want to pass a mandatory PR compliance checklist to every worker prompt
+  So that workers complete all required items before creating a PR and avoid systemic merge blockers
+
+  Background:
+    Given the implementation-supervisor.md agent definition exists
+
+  Scenario: Supervisor worker prompt includes the PR compliance checklist
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes the PR compliance checklist section
+    And the checklist is marked as MANDATORY
+
+  Scenario: Checklist item 1 — CHANGELOG.md update required
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a CHANGELOG.md checklist item
+    And the item instructs workers to add an entry under the Unreleased section
+
+  Scenario: Checklist item 2 — CONTRIBUTORS.md update required
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a CONTRIBUTORS.md checklist item
+    And the item instructs workers to add or update their contribution entry
+
+  Scenario: Checklist item 3 — commit footer required
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a commit footer checklist item
+    And the item specifies the ISSUES CLOSED footer format
+
+  Scenario: Checklist item 4 — CI must pass before PR creation
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a CI passes checklist item
+    And the item instructs workers to verify all quality gates are green
+
+  Scenario: Checklist item 5 — BDD/Behave tests required
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a BDD tests checklist item
+    And the item instructs workers to add or update Behave feature files
+
+  Scenario: Checklist item 6 — Epic reference required in PR description
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes an Epic reference checklist item
+    And the item instructs workers to reference the parent Epic issue number
+
+  Scenario: Checklist item 7 — Labels must be applied
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a labels checklist item
+    And the item instructs workers to apply labels via forgejo-label-manager
+
+  Scenario: Checklist item 8 — Milestone must be assigned
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body includes a milestone checklist item
+    And the item instructs workers to assign the earliest open milestone
+
+  Scenario: All 8 checklist items are present in the worker prompt
+    When I read the implementation supervisor agent definition
+    Then the worker prompt body contains all 8 mandatory checklist items
@@ -44,6 +44,28 @@ Feature: Session CLI commands
    When I run session CLI list with --format json
    Then the session CLI JSON list entries should match the documented contract

+  Scenario: List sessions displays full 26-character ULIDs in Rich table
+    Given there are mocked existing sessions
+    When I run session CLI list
+    Then the session CLI rich table should display full session ULIDs
+
+  Scenario: List sessions summary panel shows full ULIDs for unnamed sessions
+    Given there are mocked existing sessions
+    When I run session CLI list
+    Then the session CLI summary panel should contain full session ULIDs
+
+  Scenario: List sessions summary panel shows session names for named sessions
+    Given there are mocked existing named sessions
+    When I run session CLI list
+    Then the session CLI summary panel should show session names
+
+  Scenario: Full session ID from list output works with session tell
+    Given there are mocked existing sessions
+    When I run session CLI list
+    And I capture the first session full ULID from the output
+    And I run session CLI tell with the full session ID and prompt "Hello from list"
+    Then the session CLI tell should succeed
+
  # Show command tests
  Scenario: Show session with valid ID
    Given there is a mocked session with messages
@@ -0,0 +1,395 @@
+"""Step definitions for ACMS Index Data Model and File Traversal Engine tests."""
+
+from __future__ import annotations
+
+import tempfile
+from datetime import datetime, timedelta
+from pathlib import Path
+
+from behave import given, then, when
+
+from cleveragents.acms.index import (
+    ACMSIndex,
+    FileTraversalEngine,
+    FileType,
+    IndexEntry,
+    TierLevel,
+)
+
+
+@given("I have an ACMS index")
+def step_create_index(context):
+    """Create a new ACMS index."""
+    context.index = ACMSIndex()
+
+
+@given("I have a file traversal engine with chunk size {chunk_size:d}")
+def step_create_traversal_engine(context, chunk_size):
+    """Create a file traversal engine with specified chunk size."""
+    context.engine = FileTraversalEngine(chunk_size=chunk_size)
+
+
+@when("I create an index entry with:")
+def step_create_index_entry(context):
+    """Create an index entry from table data."""
+    data = {row["key"]: row["value"] for row in context.table}
+
+    file_type = FileType(data.get("file_type", "other"))
+    size_bytes = int(data.get("size_bytes", "0"))
+
+    context.entry = IndexEntry(
+        path=data["path"],
+        file_type=file_type,
+        size_bytes=size_bytes,
+        created_at=datetime.now(),
+        modified_at=datetime.now(),
+    )
+
+
+@then('the index entry should have path "{path}"')
+def step_check_entry_path(context, path):
+    """Verify the index entry has the expected path."""
+    assert context.entry.path == path
+
+
+@then('the index entry should have file type "{file_type}"')
+def step_check_entry_file_type(context, file_type):
+    """Verify the index entry has the expected file type."""
+    assert context.entry.file_type == FileType(file_type)
+
+
+@then("the index entry should have size {size:d} bytes")
+def step_check_entry_size(context, size):
+    """Verify the index entry has the expected size."""
+    assert context.entry.size_bytes == size
+
+
+@given('I have an index entry with path "{path}"')
+def step_create_entry_with_path(context, path):
+    """Create an index entry with a specific path."""
+    context.entry = IndexEntry(
+        path=path,
+        file_type=FileType.PYTHON,
+        size_bytes=1024,
+        created_at=datetime.now(),
+        modified_at=datetime.now(),
+    )
+
+
+@when('I add tag "{tag}" to the entry')
+def step_add_tag_to_entry(context, tag):
+    """Add a tag to the current entry."""
+    context.entry.add_tag(tag)
+
+
+@then('the entry should have tag "{tag}"')
+def step_check_entry_has_tag(context, tag):
+    """Verify the entry has a specific tag."""
+    assert context.entry.has_tag(tag)
+
+
+@then("the entry should have {count:d} tags")
+def step_check_entry_tag_count(context, count):
+    """Verify the entry has the expected number of tags."""
+    assert len(context.entry.tags) == count
+
+
+@when('I set the tier level to "{tier}"')
+def step_set_entry_tier(context, tier):
+    """Set the tier level for the entry."""
+    context.entry.set_tier(TierLevel(tier))
+
+
+@then('the entry should have tier level "{tier}"')
+def step_check_entry_tier(context, tier):
+    """Verify the entry has the expected tier level."""
+    assert context.entry.tier == TierLevel(tier)
+
+
+@when("I add the entry to the index")
+def step_add_entry_to_index(context):
+    """Add the current entry to the index."""
+    context.index.add_entry(context.entry)
+
+
+@then("the index should contain {count:d} entry")
+def step_check_index_entry_count_singular(context, count):
+    """Verify the index has the expected number of entries."""
+    assert context.index.get_entry_count() == count
+
+
+@then("the index should contain {count:d} entries")
+def step_check_index_entry_count(context, count):
+    """Verify the index has the expected number of entries."""
+    assert context.index.get_entry_count() == count
+
+
+@then('I should be able to retrieve the entry by path "{path}"')
+def step_retrieve_entry_by_path(context, path):
+    """Verify we can retrieve an entry by path."""
+    entry = context.index.get_entry(path)
+    assert entry is not None
+    assert entry.path == path
+
+
+@given("I have an index with entries:")
+def step_create_index_with_entries(context):
+    """Create an index with multiple entries from table data."""
+    context.index = ACMSIndex()
+
+    for row in context.table:
+        path = row["path"]
+        file_type = FileType(row.get("file_type", "other"))
+
+        entry = IndexEntry(
+            path=path,
+            file_type=file_type,
+            size_bytes=1024,
+            created_at=datetime.now(),
+            modified_at=datetime.now(),
+        )
+
+        # Add tags if present
+        if "tags" in row:
+            for tag in row["tags"].split(","):
+                entry.add_tag(tag.strip())
+
+        # Set tier if present
+        if "tier" in row:
+            entry.set_tier(TierLevel(row["tier"]))
+
+        context.index.add_entry(entry)
+
+
+@given("I have an index with {count:d} entries")
+def step_create_index_with_n_entries(context, count):
+    """Create an index with a specified number of entries."""
+    context.index = ACMSIndex()
+    for i in range(count):
+        entry = IndexEntry(
+            path=f"/project/file{i}.py",
+            file_type=FileType.PYTHON,
+            size_bytes=1024,
+            created_at=datetime.now(),
+            modified_at=datetime.now(),
+        )
+        context.index.add_entry(entry)
+
+
+@when('I query the index by path pattern "{pattern}"')
+def step_query_by_path_pattern(context, pattern):
+    """Query the index by path pattern."""
+    context.query_results = context.index.query_by_path(pattern)
+
+
+@then("I should get {count:d} results")
+def step_check_query_result_count(context, count):
+    """Verify the query returned the expected number of results."""
+    assert len(context.query_results) == count, (
+        f"Expected {count} results, got {len(context.query_results)}"
+    )
+
+
+@then("I should get {count:d} result")
+def step_check_query_result_count_singular(context, count):
+    """Verify the query returned the expected number of results (singular)."""
+    assert len(context.query_results) == count, (
+        f"Expected {count} result, got {len(context.query_results)}"
+    )
+
+
+@then('the results should include "{path}"')
+def step_check_result_includes_path(context, path):
+    """Verify the query results include a specific path."""
+    paths = [entry.path for entry in context.query_results]
+    assert path in paths
+
+
+@when('I query the index by file type "{file_type}"')
+def step_query_by_file_type(context, file_type):
+    """Query the index by file type."""
+    context.query_results = context.index.query_by_type(FileType(file_type))
+
+
+@then('all results should have file type "{file_type}"')
+def step_check_all_results_file_type(context, file_type):
+    """Verify all results have the expected file type."""
+    expected_type = FileType(file_type)
+    for entry in context.query_results:
+        assert entry.file_type == expected_type
+
+
+@when('I query the index by tag "{tag}"')
+def step_query_by_tag(context, tag):
+    """Query the index by tag."""
+    context.query_results = context.index.query_by_tag(tag)
+
+
+@when('I query the index by tier level "{tier}"')
+def step_query_by_tier(context, tier):
+    """Query the index by tier level."""
+    context.query_results = context.index.query_by_tier(TierLevel(tier))
+
+
+@then('the result should have path "{path}"')
+def step_check_single_result_path(context, path):
+    """Verify the single result has the expected path."""
+    assert len(context.query_results) == 1
+    assert context.query_results[0].path == path
+
+
+@given("I have an index with entries from different dates")
+def step_create_index_with_dated_entries(context):
+    """Create an index with entries from different dates."""
+    context.index = ACMSIndex()
+
+    now = datetime.now()
+    dates = [
+        now - timedelta(days=10),
+        now - timedelta(days=5),
+        now - timedelta(days=1),
+        now,
+    ]
+
+    for i, date in enumerate(dates):
+        entry = IndexEntry(
+            path=f"/project/file{i}.py",
+            file_type=FileType.PYTHON,
+            size_bytes=1024,
+            created_at=date,
+            modified_at=date,
+        )
+        context.index.add_entry(entry)
+
+
+@when('I query the index for entries modified after "{date_str}"')
+def step_query_by_recency(context, date_str):
+    """Query the index for entries modified after a specific date."""
+    # Parse date string (format: YYYY-MM-DD)
+    date = datetime.strptime(date_str, "%Y-%m-%d")
+    context.query_results = context.index.query_by_recency(date)
+
+
+@then("I should get entries modified after that date")
+def step_check_recency_results(context):
+    """Verify the recency query returned valid results."""
+    assert len(context.query_results) > 0
+
+
+@given("I have a test directory with {count:d} files")
+def step_create_test_directory(context, count):
+    """Create a temporary directory with test files."""
+    context.temp_dir = tempfile.TemporaryDirectory()
+    temp_path = Path(context.temp_dir.name)
+
+    # Create subdirectories and files
+    for i in range(count):
+        subdir = temp_path / f"subdir{i % 10}"
+        subdir.mkdir(exist_ok=True)
+
+        file_path = subdir / f"file{i}.py"
+        file_path.write_text(f"# Test file {i}\nprint('Hello {i}')\n")
+
+
+@when("I traverse and index the directory")
+def step_traverse_and_index(context):
+    """Traverse and index the test directory."""
+    context.engine.reset_index()
+    context.index = context.engine.traverse_and_index(context.temp_dir.name)
+
+
+@when("I traverse and index the directory with chunk size {chunk_size:d}")
+def step_traverse_and_index_with_chunk_size(context, chunk_size):
+    """Traverse and index the test directory with a specific chunk size."""
+    engine = FileTraversalEngine(chunk_size=chunk_size)
+    context.index = engine.traverse_and_index(context.temp_dir.name)
+
+
+@then("all entries should have valid file paths")
+def step_check_valid_file_paths(context):
+    """Verify all entries have valid file paths."""
+    for entry in context.index.get_all_entries():
+        assert entry.path
+        assert len(entry.path) > 0
+
+
+@then("the traversal should complete without timeout")
+def step_check_no_timeout(context):
+    """Verify the traversal completed without timeout."""
+    # This step passes if we got here without timing out
+    assert True
+
+
+@given("I have a test directory with files including:")
+def step_create_test_directory_with_specific_files(context):
+    """Create a test directory with specific files."""
+    context.temp_dir = tempfile.TemporaryDirectory()
+    temp_path = Path(context.temp_dir.name)
+
+    for row in context.table:
+        file_path = temp_path / row["path"].lstrip("/")
+        file_path.parent.mkdir(parents=True, exist_ok=True)
+        file_path.write_text("test content")
+
+
+@when('I traverse and index the directory excluding "{exclude1}" and "{exclude2}"')
+def step_traverse_with_exclusions(context, exclude1, exclude2):
+    """Traverse and index with exclusion patterns."""
+    context.engine.reset_index()
+    context.index = context.engine.traverse_and_index(
+        context.temp_dir.name,
+        exclude_patterns=[exclude1, exclude2],
+    )
+
+
+@then('the index should not contain "{pattern}" paths')
+def step_check_no_excluded_paths(context, pattern):
+    """Verify the index doesn't contain paths matching the pattern."""
+    for entry in context.index.get_all_entries():
+        assert pattern not in entry.path
+
+
+@when("I get all entries from the index")
+def step_get_all_entries(context):
+    """Get all entries from the index."""
+    context.query_results = context.index.get_all_entries()
+
+
+@when("I get the entry count")
+def step_get_entry_count(context):
+    """Get the entry count from the index."""
+    context.entry_count = context.index.get_entry_count()
+
+
+@then("the count should be {count:d}")
+def step_check_entry_count_value(context, count):
+    """Verify the entry count matches the expected value."""
+    assert context.entry_count == count
+
+
+@when("I remove an entry by path")
+def step_remove_entry(context):
+    """Remove an entry from the index."""
+    # Get the first entry's path
+    entries = context.index.get_all_entries()
+    if entries:
+        context.index.remove_entry(entries[0].path)
+
+
+@when("I query the index with filters:")
+def step_query_with_combined_filters(context):
+    """Query the index with multiple filters."""
+    filters = {row["filter"]: row["value"] for row in context.table}
+
+    path_pattern = filters.get("path_pattern")
+    file_type_str = filters.get("file_type")
+    tier_str = filters.get("tier")
+
+    file_type = FileType(file_type_str) if file_type_str else None
+    tier = TierLevel(tier_str) if tier_str else None
+
+    context.query_results = context.index.query_combined(
+        path_pattern=path_pattern,
+        file_type=file_type,
+        tier=tier,
+    )
@@ -105,6 +105,36 @@ def step_create_json_file_with_name(context, name):
    context.import_file.write_text(json.dumps(data, indent=2), encoding="utf-8")


+# ---------------------------------------------------------------------------
+# When — list
+# ---------------------------------------------------------------------------
+
+
+@when('I run actor context list "{pattern}"')
+def step_list_with_pattern(context, pattern):
+    context.result = context.runner.invoke(
+        actor_context_app,
+        [
+            "list",
+            pattern,
+            "--context-dir",
+            str(context.context_dir),
+        ],
+    )
+
+
+@when("I run actor context list without a pattern")
+def step_list_without_pattern(context):
+    context.result = context.runner.invoke(
+        actor_context_app,
+        [
+            "list",
+            "--context-dir",
+            str(context.context_dir),
+        ],
+    )
+
+
 # ---------------------------------------------------------------------------
 # When — clear
 # ---------------------------------------------------------------------------
@@ -383,6 +413,15 @@ def step_roundtrip_import(context, name):
 # ---------------------------------------------------------------------------


+@then("the actor context list command should succeed")
+def step_list_success(context):
+    assert context.result.exit_code == 0, (
+        f"Expected exit 0, got {context.result.exit_code}.\n"
+        f"stdout: {context.result.output}\n"
+        f"stderr: {getattr(context.result, 'stderr', '')}"
+    )
+
+
@then("the actor context clear command should succeed")
 def step_clear_success(context):
    assert context.result.exit_code == 0, (
@@ -514,6 +553,18 @@ def step_all_contexts_empty(context):
 # ---------------------------------------------------------------------------


+@then('the list output should contain "{text}"')
+def step_list_output_contains(context, text):
+    output = context.result.output
+    assert text in output, f"Expected '{text}' in list output: {output}"
+
+
+@then('the list output should not contain "{text}"')
+def step_list_output_not_contains(context, text):
+    output = context.result.output
+    assert text not in output, f"Did not expect '{text}' in list output: {output}"
+
+
@then('the output should contain valid JSON with key "{key}"')
 def step_output_json_key(context, key):
    output = context.result.output
@@ -809,6 +809,9 @@ class _BrokenSession:
    def rollback(self) -> None:
        pass

+    def close(self) -> None:
+        pass
+
    def query(self, *_args: Any, **_kwargs: Any) -> Any:
        raise SQLAlchemyDatabaseError("mock", {}, Exception("broken"))

@@ -985,8 +988,10 @@ def step_save_with_spy(context: Context) -> None:
    object.__setattr__(real_session, "flush", spy_flush)
    object.__setattr__(real_session, "commit", spy_commit)

+    # Pass the session explicitly to test the UoW path: save() must flush
+    # but must NOT commit (the caller owns the transaction boundary).
    repo = LLMTraceRepository(session_factory=lambda: real_session)
-    repo.save(context.trace)
+    repo.save(context.trace, session=real_session)
    # Commit so the data is visible for subsequent queries
    object.__setattr__(real_session, "commit", original_commit)
    real_session.commit()
@@ -1030,7 +1035,9 @@ def step_save_in_uow_rollback(context: Context) -> None:
    session = context.uow_session_factory()
    repo = LLMTraceRepository(session_factory=lambda: session)
    try:
-        repo.save(context.trace)
+        # Pass the session explicitly to use UoW mode: save() flushes but
+        # does NOT commit, so the caller's rollback can undo the change.
+        repo.save(context.trace, session=session)
        # Simulate a subsequent failure that triggers rollback
        raise RuntimeError("Simulated failure after save")
    except RuntimeError:
@@ -316,6 +316,17 @@ def step_then_temp_connection_closed(context) -> None:
    assert context.current_rev_fake_engine.connections[0].exit_called is True


+@then("the SQLite engine for get_current_revision should use check_same_thread=False")
+def step_then_get_current_revision_check_same_thread(context) -> None:
+    _url, kwargs = context.current_rev_create_call
+    assert "connect_args" in kwargs, (
+        "Expected connect_args to be passed to create_engine for SQLite"
+    )
+    assert kwargs["connect_args"].get("check_same_thread") is False, (
+        "Expected check_same_thread=False in connect_args for SQLite engine"
+    )
+
+
@when("I initialize or upgrade a file-based SQLite database")
 def step_when_init_file_based_sqlite(context) -> None:
    import shutil
@@ -0,0 +1,185 @@
+"""Step definitions for PR compliance checklist in implementation supervisor."""
+
+from pathlib import Path
+from typing import Any
+
+from behave import given, then, when
+
+PROJECT_ROOT = Path(__file__).resolve().parents[3]
+AGENT_DEF_PATH = PROJECT_ROOT / ".opencode" / "agents" / "implementation-supervisor.md"
+
+
+@given("the implementation-supervisor.md agent definition exists")
+def step_agent_def_exists(context: Any) -> None:
+    """Verify the implementation supervisor agent definition file exists."""
+    assert AGENT_DEF_PATH.exists(), f"Agent definition not found at {AGENT_DEF_PATH}"
+    context.agent_def_path = AGENT_DEF_PATH
+
+
+@when("I read the implementation supervisor agent definition")
+def step_read_agent_def(context: Any) -> None:
+    """Read the implementation supervisor agent definition."""
+    context.agent_def_content = AGENT_DEF_PATH.read_text(encoding="utf-8")
+
+
+@then("the worker prompt body includes the PR compliance checklist section")
+def step_prompt_includes_checklist(context: Any) -> None:
+    """Verify the worker prompt body includes the PR compliance checklist."""
+    assert "PR Compliance Checklist" in context.agent_def_content, (
+        "Worker prompt body does not include 'PR Compliance Checklist'"
+    )
+
+
+@then("the checklist is marked as MANDATORY")
+def step_checklist_is_mandatory(context: Any) -> None:
+    """Verify the checklist is marked as MANDATORY."""
+    assert "MANDATORY" in context.agent_def_content, (
+        "PR Compliance Checklist is not marked as MANDATORY"
+    )
+
+
+@then("the worker prompt body includes a CHANGELOG.md checklist item")
+def step_prompt_includes_changelog_item(context: Any) -> None:
+    """Verify the worker prompt body includes a CHANGELOG.md checklist item."""
+    assert "CHANGELOG.md" in context.agent_def_content, (
+        "Worker prompt body does not include a CHANGELOG.md checklist item"
+    )
+
+
+@then("the item instructs workers to add an entry under the Unreleased section")
+def step_changelog_item_unreleased(context: Any) -> None:
+    """Verify the CHANGELOG.md item mentions the Unreleased section."""
+    assert "[Unreleased]" in context.agent_def_content, (
+        "CHANGELOG.md checklist item does not mention the [Unreleased] section"
+    )
+
+
+@then("the worker prompt body includes a CONTRIBUTORS.md checklist item")
+def step_prompt_includes_contributors_item(context: Any) -> None:
+    """Verify the worker prompt body includes a CONTRIBUTORS.md checklist item."""
+    assert "CONTRIBUTORS.md" in context.agent_def_content, (
+        "Worker prompt body does not include a CONTRIBUTORS.md checklist item"
+    )
+
+
+@then("the item instructs workers to add or update their contribution entry")
+def step_contributors_item_add_update(context: Any) -> None:
+    """Verify the CONTRIBUTORS.md item instructs workers to add or update."""
+    assert "add or update" in context.agent_def_content, (
+        "CONTRIBUTORS.md checklist item does not instruct workers to add or update"
+    )
+
+
+@then("the worker prompt body includes a commit footer checklist item")
+def step_prompt_includes_commit_footer_item(context: Any) -> None:
+    """Verify the worker prompt body includes a commit footer checklist item."""
+    assert "Commit footer" in context.agent_def_content, (
+        "Worker prompt body does not include a commit footer checklist item"
+    )
+
+
+@then("the item specifies the ISSUES CLOSED footer format")
+def step_commit_footer_issues_closed(context: Any) -> None:
+    """Verify the commit footer item specifies the ISSUES CLOSED format."""
+    assert "ISSUES CLOSED" in context.agent_def_content, (
+        "Commit footer checklist item does not specify the ISSUES CLOSED format"
+    )
+
+
+@then("the worker prompt body includes a CI passes checklist item")
+def step_prompt_includes_ci_item(context: Any) -> None:
+    """Verify the worker prompt body includes a CI passes checklist item."""
+    assert "CI passes" in context.agent_def_content, (
+        "Worker prompt body does not include a CI passes checklist item"
+    )
+
+
+@then("the item instructs workers to verify all quality gates are green")
+def step_ci_item_quality_gates(context: Any) -> None:
+    """Verify the CI item instructs workers to verify quality gates."""
+    assert "quality gates" in context.agent_def_content, (
+        "CI checklist item does not mention quality gates"
+    )
+
+
+@then("the worker prompt body includes a BDD tests checklist item")
+def step_prompt_includes_bdd_item(context: Any) -> None:
+    """Verify the worker prompt body includes a BDD/Behave tests checklist item."""
+    assert "BDD/Behave tests" in context.agent_def_content, (
+        "Worker prompt body does not include a BDD/Behave tests checklist item"
+    )
+
+
+@then("the item instructs workers to add or update Behave feature files")
+def step_bdd_item_feature_files(context: Any) -> None:
+    """Verify the BDD item instructs workers to add or update feature files."""
+    assert "added or updated" in context.agent_def_content, (
+        "BDD checklist item does not instruct workers to add or update feature files"
+    )
+
+
+@then("the worker prompt body includes an Epic reference checklist item")
+def step_prompt_includes_epic_item(context: Any) -> None:
+    """Verify the worker prompt body includes an Epic reference checklist item."""
+    assert "Epic reference" in context.agent_def_content, (
+        "Worker prompt body does not include an Epic reference checklist item"
+    )
+
+
+@then("the item instructs workers to reference the parent Epic issue number")
+def step_epic_item_parent_reference(context: Any) -> None:
+    """Verify the Epic item instructs workers to reference the parent Epic."""
+    assert "parent Epic" in context.agent_def_content, (
+        "Epic checklist item does not instruct workers to reference the parent Epic"
+    )
+
+
+@then("the worker prompt body includes a labels checklist item")
+def step_prompt_includes_labels_item(context: Any) -> None:
+    """Verify the worker prompt body includes a labels checklist item."""
+    assert "Labels" in context.agent_def_content, (
+        "Worker prompt body does not include a labels checklist item"
+    )
+
+
+@then("the item instructs workers to apply labels via forgejo-label-manager")
+def step_labels_item_forgejo_label_manager(context: Any) -> None:
+    """Verify the labels item instructs workers to use forgejo-label-manager."""
+    assert "forgejo-label-manager" in context.agent_def_content, (
+        "Labels checklist item does not mention forgejo-label-manager"
+    )
+
+
+@then("the worker prompt body includes a milestone checklist item")
+def step_prompt_includes_milestone_item(context: Any) -> None:
+    """Verify the worker prompt body includes a milestone checklist item."""
+    assert "Milestone" in context.agent_def_content, (
+        "Worker prompt body does not include a milestone checklist item"
+    )
+
+
+@then("the item instructs workers to assign the earliest open milestone")
+def step_milestone_item_earliest(context: Any) -> None:
+    """Verify the milestone item instructs workers to assign the earliest open milestone."""
+    assert "earliest open milestone" in context.agent_def_content, (
+        "Milestone checklist item does not mention the earliest open milestone"
+    )
+
+
+@then("the worker prompt body contains all 8 mandatory checklist items")
+def step_prompt_contains_all_8_items(context: Any) -> None:
+    """Verify the worker prompt body contains all 8 mandatory checklist items."""
+    required_items = [
+        "CHANGELOG.md",
+        "CONTRIBUTORS.md",
+        "ISSUES CLOSED",
+        "CI passes",
+        "BDD/Behave tests",
+        "Epic reference",
+        "forgejo-label-manager",
+        "earliest open milestone",
+    ]
+    missing = [item for item in required_items if item not in context.agent_def_content]
+    assert not missing, (
+        f"Worker prompt body is missing the following checklist items: {missing}"
+    )
@@ -4,6 +4,7 @@ from __future__ import annotations

 import json
 import os
+import re
 import tempfile
 from datetime import datetime
 from typing import Any
@@ -160,14 +161,34 @@ def step_existing_sessions(context: Context) -> None:
    context.mock_service.list.return_value = sessions


+@given("there are mocked existing named sessions")
+def step_existing_named_sessions(context: Context) -> None:
+    """Set up mocked sessions with names for Summary panel name-display test."""
+    sessions = [
+        _make_session(
+            session_id=_SESSION_ID,
+            actor_name="openai/gpt-4",
+            messages=[_make_message(sequence=0)],
+        ),
+        _make_session(session_id=_SESSION_ID_2),
+    ]
+    sessions[0].name = "weekly-planning"
+    sessions[1].name = "refactor-sprint"
+    context.mock_service.list.return_value = sessions
+
+
@when("I run session CLI list")
 def step_list(context: Context) -> None:
-    context.result = context.runner.invoke(session_app, ["list"])
+    context.result = context.runner.invoke(
+        session_app, ["list"], env={"COLUMNS": "200"}
+    )


@when("I run session CLI list with --format json")
 def step_list_json(context: Context) -> None:
-    context.result = context.runner.invoke(session_app, ["list", "--format", "json"])
+    context.result = context.runner.invoke(
+        session_app, ["list", "--format", "json"], env={"COLUMNS": "200"}
+    )


@then("the session CLI should show all sessions in a table")
@@ -176,6 +197,125 @@ def step_list_shows_table(context: Context) -> None:
    assert "Sessions" in context.result.output


+@then("the session CLI rich table should display full session ULIDs")
+def step_list_rich_table_full_ulids(context: Context) -> None:
+    """Verify the Rich table displays the full 26-character session ULIDs."""
+    assert context.result.exit_code == 0
+    output = context.result.output
+    # Restrict assertion to the table region (before the Summary panel) so
+    # that a regression back to 8-char truncation is not masked by the full
+    # ULID appearing elsewhere (e.g. in the Summary panel).
+    summary_idx = output.find("Summary")
+    table_output = output[:summary_idx] if summary_idx != -1 else output
+    assert _SESSION_ID in table_output, (
+        f"Full ULID {_SESSION_ID} not found in Rich table output:\n"
+        f"{context.result.output}"
+    )
+    assert _SESSION_ID_2 in table_output, (
+        f"Full ULID {_SESSION_ID_2} not found in Rich table output:\n"
+        f"{context.result.output}"
+    )
+    # Negative guard: ensure the table contains full 26-character ULIDs,
+    # not the old 8-character truncated form.  A standalone 8-char prefix
+    # (not as part of the full ULID) would indicate the [:8] slice was not
+    # removed.
+    table_without_full_ids = table_output.replace(_SESSION_ID, "").replace(
+        _SESSION_ID_2, ""
+    )
+    assert _SESSION_ID[:8] not in table_without_full_ids, (
+        f"Truncated 8-char ID {_SESSION_ID[:8]} found in Rich table output:\n"
+        f"{context.result.output}"
+    )
+    assert _SESSION_ID_2[:8] not in table_without_full_ids, (
+        f"Truncated 8-char ID {_SESSION_ID_2[:8]} found in Rich table output:\n"
+        f"{context.result.output}"
+    )
+
+
+@then("the session CLI summary panel should contain full session ULIDs")
+def step_list_summary_full_ulids(context: Context) -> None:
+    """Verify the Summary panel shows full ULIDs for unnamed sessions."""
+    assert context.result.exit_code == 0
+    output = context.result.output
+    assert "Summary" in output, f"Summary panel not found in output:\n{output}"
+    # Extract the Summary panel region to avoid a false pass from the Rich
+    # table also containing the same full ULIDs.
+    summary_idx = output.find("Summary")
+    summary_output = output[summary_idx:]
+    # Both Most Recent and Oldest entries should display full ULIDs when
+    # sessions are unnamed.  Asserting only one ID would allow a regression
+    # that re-introduced [:8] on one fallback path while keeping the other
+    # intact to pass the test undetected.
+    assert _SESSION_ID in summary_output, (
+        f"Full ULID {_SESSION_ID} not found in Summary panel:\n{output}"
+    )
+    assert _SESSION_ID_2 in summary_output, (
+        f"Full ULID {_SESSION_ID_2} not found in Summary panel:\n{output}"
+    )
+
+
+@then("the session CLI summary panel should show session names")
+def step_list_summary_shows_names(context: Context) -> None:
+    """Verify the Summary panel shows session names (not ULIDs) for named sessions."""
+    assert context.result.exit_code == 0
+    output = context.result.output
+    summary_idx = output.find("Summary")
+    assert summary_idx != -1, f"Summary panel not found in output:\n{output}"
+    summary_output = output[summary_idx:]
+    assert "weekly-planning" in summary_output, (
+        f"'weekly-planning' not found in Summary panel:\n{output}"
+    )
+    assert "refactor-sprint" in summary_output, (
+        f"'refactor-sprint' not found in Summary panel:\n{output}"
+    )
+    # The Summary should NOT show the raw ULID when session names are present
+    assert _SESSION_ID not in summary_output, (
+        f"Full ULID {_SESSION_ID} unexpectedly found in Summary panel:\n{output}"
+    )
+    assert _SESSION_ID_2 not in summary_output, (
+        f"Full ULID {_SESSION_ID_2} unexpectedly found in Summary panel:\n{output}"
+    )
+
+
+@when("I capture the first session full ULID from the output")
+def step_capture_first_ulid(context: Context) -> None:
+    """Parse the first session's full ULID from the output and store it
+    for subsequent steps (round-trip tell test)."""
+    assert context.result.exit_code == 0
+    output = context.result.output
+    # Restrict the search to the table region (before the Summary panel)
+    # so that a ULID appearing in the Summary panel is not accidentally
+    # captured as the "first" session ID.
+    summary_idx = output.find("Summary")
+    search_region = output[:summary_idx] if summary_idx != -1 else output
+    match = re.search(r"[0-9A-HJKMNP-TV-Z]{26}", search_region)
+    assert match is not None, (
+        f"No 26-character ULID found in table output:\n{search_region}"
+    )
+    context.full_session_id = match.group()
+    assert len(context.full_session_id) == 26, (
+        f"Parsed ULID '{context.full_session_id}' is not 26 characters"
+    )
+    # Sanity check: the captured ID should match one of the known fixture IDs
+    assert context.full_session_id in (_SESSION_ID, _SESSION_ID_2), (
+        f"Captured ULID '{context.full_session_id}' does not match any fixture ID"
+    )
+
+
+@when('I run session CLI tell with the full session ID and prompt "{prompt}"')
+def step_tell_with_stored_ulid(context: Context, prompt: str) -> None:
+    """Run session tell using the full ULID stored from session list."""
+    session_id = context.full_session_id
+    context.mock_service.append_message.side_effect = [
+        _make_message(MessageRole.USER, prompt, 0),
+        _make_message(MessageRole.ASSISTANT, f"Acknowledged: {prompt}", 1),
+    ]
+    context.result = context.runner.invoke(
+        session_app,
+        ["tell", "--session", session_id, prompt],
+    )
+
+
 # ---------------------------------------------------------------------------
 # Show
 # ---------------------------------------------------------------------------
@@ -0,0 +1,85 @@
+"""Steps for TDD Issue #10507 — get_current_revision() SQLite threading fix."""
+
+from __future__ import annotations
+
+from typing import Any
+from unittest.mock import MagicMock, patch
+
+from behave import then, when
+
+
+def _run_get_current_revision_and_capture_kwargs(
+    context: Any,
+) -> None:
+    """Shared helper: call get_current_revision() and capture create_engine kwargs.
+
+    Mocks ``create_engine`` and ``MigrationContext.configure`` so the call
+    completes without a real database.  The keyword arguments passed to
+    ``create_engine`` are stored on ``context.engine_creation_kwargs`` for
+    subsequent assertion steps.
+    """
+    fake_engine = MagicMock()
+    fake_connection = MagicMock()
+    fake_connection.__enter__ = MagicMock(return_value=fake_connection)
+    fake_connection.__exit__ = MagicMock(return_value=False)
+    fake_engine.connect.return_value = fake_connection
+
+    migration_ctx = MagicMock()
+    migration_ctx.get_current_revision.return_value = None
+
+    captured_kwargs: list[dict[str, Any]] = []
+
+    def fake_create_engine(url: str, **kwargs: Any) -> MagicMock:
+        captured_kwargs.append(kwargs)
+        return fake_engine
+
+    with (
+        patch(
+            "cleveragents.infrastructure.database.migration_runner.create_engine",
+            side_effect=fake_create_engine,
+        ),
+        patch(
+            "cleveragents.infrastructure.database.migration_runner.MigrationContext.configure",
+            return_value=migration_ctx,
+        ),
+    ):
+        context.revision_result = context.runner.get_current_revision()
+
+    context.engine_creation_kwargs = captured_kwargs
+
+
+@when("I request the current revision and capture the engine creation args")
+def step_when_capture_engine_args(context: Any) -> None:
+    """Call get_current_revision and capture the create_engine call arguments."""
+    _run_get_current_revision_and_capture_kwargs(context)
+
+
+@then("the SQLite engine should be created with check_same_thread set to False")
+def step_then_sqlite_engine_has_check_same_thread(context: Any) -> None:
+    """Verify the SQLite engine was created with check_same_thread=False."""
+    assert len(context.engine_creation_kwargs) == 1, (
+        f"Expected exactly 1 create_engine call, got {len(context.engine_creation_kwargs)}"
+    )
+    kwargs = context.engine_creation_kwargs[0]
+    assert "connect_args" in kwargs, (
+        "Expected connect_args in create_engine kwargs for SQLite, "
+        f"but got kwargs: {kwargs}"
+    )
+    assert kwargs["connect_args"].get("check_same_thread") is False, (
+        "Expected check_same_thread=False in connect_args, "
+        f"but got: {kwargs['connect_args']}"
+    )
+
+
+@then("the non-SQLite engine should be created without check_same_thread")
+def step_then_non_sqlite_engine_no_check_same_thread(context: Any) -> None:
+    """Verify non-SQLite engines are not given check_same_thread."""
+    assert len(context.engine_creation_kwargs) == 1, (
+        f"Expected exactly 1 create_engine call, got {len(context.engine_creation_kwargs)}"
+    )
+    kwargs = context.engine_creation_kwargs[0]
+    connect_args = kwargs.get("connect_args", {})
+    assert "check_same_thread" not in connect_args, (
+        "Expected check_same_thread to be absent for non-SQLite engine, "
+        f"but got connect_args: {connect_args}"
+    )
@@ -0,0 +1,31 @@
+@tdd_issue @tdd_issue_10507
+Feature: TDD Issue #10507 — get_current_revision() must pass check_same_thread=False for SQLite
+  As a developer using MigrationRunner in a multi-threaded application
+  I want get_current_revision() to work safely from background threads
+  So that async startup flows and background migration checks do not crash
+
+  The root cause is that MigrationRunner.get_current_revision() calls
+  create_engine(self.database_url) without connect_args={"check_same_thread": False}
+  for SQLite databases.  When called from a thread other than the one that
+  created the engine, SQLite raises:
+      ProgrammingError: SQLite objects created in a thread can only be
+      used in that same thread.
+
+  The sibling method init_or_upgrade() already passes check_same_thread=False
+  for SQLite, making this an inconsistency in the same class.  Because
+  get_pending_migrations() and check_migrations_needed() both delegate to
+  get_current_revision(), the threading bug propagates to all three methods.
+
+  The fix adds connect_args={"check_same_thread": False} to the create_engine()
+  call in get_current_revision() when the database URL starts with "sqlite",
+  consistent with the existing pattern in init_or_upgrade().
+
+  Scenario: get_current_revision passes check_same_thread=False for SQLite engine
+    Given a migration runner configured for "sqlite:///:memory:"
+    When I request the current revision and capture the engine creation args
+    Then the SQLite engine should be created with check_same_thread set to False
+
+  Scenario: get_current_revision does not pass check_same_thread for non-SQLite engine
+    Given a migration runner configured for "postgresql://user:pass@localhost/testdb"
+    When I request the current revision and capture the engine creation args
+    Then the non-SQLite engine should be created without check_same_thread
@@ -61,7 +61,6 @@ nav:
    - Reference/Command Input & Sessions: tui/input-and-sessions.md
    - Configuration, Key Bindings & Integration: tui/configuration-and-integration.md
  - FAQ: faq.md
-  - Quick Start: quickstart.md
  - Changelog: CHANGELOG.md
  - Contributing: CONTRIBUTING.md
  - Reference: reference/
@@ -94,7 +93,7 @@ nav:
    - ADR-025 Observability & Logging: adr/ADR-025-observability-and-logging.md
    - ADR-026 Agent-to-Agent Protocol (A2A): adr/ADR-026-agent-client-protocol.md
    - ADR-027 Language Server Protocol (LSP) Integration: adr/ADR-027-language-server-protocol.md
-    - ADR-028 Skill/Tool Abstraction Definition: adr/ADR-028-agent-skills-standard.md
+    - ADR-028 Agent Skills Standard (AgentSkills.io): adr/ADR-028-agent-skills-standard.md
    - ADR-029 Model Context Protocol (MCP) Adoption: adr/ADR-029-model-context-protocol.md
    - ADR-030 Skill Abstraction Definition: adr/ADR-030-skill-abstraction-definition.md
    - ADR-031 Actor Abstraction Definition: adr/ADR-031-actor-abstraction-definition.md
@@ -81,6 +81,38 @@ List Context Files

    [Teardown]    Cleanup Test Directory

+Actor Context List Supports Regex Filter
+    [Documentation]    Verify actor context list supports optional regex filtering
+    ...                Uses --context-dir to point at a directory of named actor
+    ...                context subdirectories so the test is self-contained and
+    ...                does not depend on project-context state.
+    [Setup]    Setup Test Directory
+
+    # Create a temporary context directory with two named actor context subdirs
+    ${ctx_dir}=    Set Variable    ${TEST_DIR}${/}actor_contexts
+    Create Directory    ${ctx_dir}
+    Create Directory    ${ctx_dir}${/}test-context
+    Create Directory    ${ctx_dir}${/}utils-context
+
+    # Filter by "test" — should match test-context but not utils-context
+    ${result}=    Run Process    ${PYTHON}    -m    cleveragents    actor    context    list    test
+    ...    --context-dir    ${ctx_dir}
+    ...    cwd=${TEST_DIR}    timeout=120s    on_timeout=kill
+
+    Should Be Equal As Integers    ${result.rc}    0
+    Should Contain    ${result.stdout}    test-context
+    Should Not Contain    ${result.stdout}    utils-context
+
+    # Filter by "nomatch" — no contexts match, should print "No contexts found."
+    ${result}=    Run Process    ${PYTHON}    -m    cleveragents    actor    context    list    nomatch
+    ...    --context-dir    ${ctx_dir}
+    ...    cwd=${TEST_DIR}    timeout=120s    on_timeout=kill
+
+    Should Be Equal As Integers    ${result.rc}    0
+    Should Contain    ${result.stdout}    No contexts found.
+
+    [Teardown]    Cleanup Test Directory
+
 Show Context Content
    [Documentation]    Test context show command
    [Setup]    Initialize Test Project With Context
@@ -5,12 +5,22 @@ technology-specific vocabulary extensions, and the DetailLevelMap
 inheritance mechanism for resolving named detail levels across the
 ontology hierarchy (Layer 3 -> Layer 2 -> Layer 1 -> Layer 0).

+Also provides the ACMS index data model and file traversal engine for
+indexing large projects.
+
 Based on ``docs/specification.md`` ~lines 42333-42422, 44405-44420.
 """

 from __future__ import annotations

 from cleveragents.acms import uko as _uko
+from cleveragents.acms.index import (
+    ACMSIndex,
+    FileTraversalEngine,
+    FileType,
+    IndexEntry,
+    TierLevel,
+)
 from cleveragents.acms.uko import (
    CODE_DETAIL_LEVEL_MAP,
    FUNC_DETAIL_LEVEL_MAP,
@@ -62,6 +72,14 @@ from cleveragents.acms.uko import (
    resolve_detail_level,
 )

-# Re-export everything published by the ``uko`` sub-package so the two
-# ``__all__`` lists stay in sync automatically.
-__all__: list[str] = list(_uko.__all__)
+# Combine exports from both uko and index modules
+_uko_exports = list(_uko.__all__)
+_index_exports = [
+    "ACMSIndex",
+    "FileTraversalEngine",
+    "FileType",
+    "IndexEntry",
+    "TierLevel",
+]
+
+__all__: list[str] = _uko_exports + _index_exports
@@ -0,0 +1,412 @@
+"""ACMS Index Data Model and File Traversal Engine.
+
+Provides the foundational data model for indexed context entries and a
+file traversal engine that can handle 10,000+ files without timeout using
+chunked processing.
+
+Based on issue #9579 and ``docs/specification.md`` ~lines 44405-44420.
+"""
+
+from __future__ import annotations
+
+from collections.abc import Iterator
+from dataclasses import dataclass, field
+from datetime import datetime
+from enum import StrEnum
+from pathlib import Path
+
+
+class FileType(StrEnum):
+    """File type enumeration for index entries."""
+
+    PYTHON = "python"
+    JAVASCRIPT = "javascript"
+    TYPESCRIPT = "typescript"
+    JAVA = "java"
+    RUST = "rust"
+    MARKDOWN = "markdown"
+    TEXT = "text"
+    JSON = "json"
+    YAML = "yaml"
+    XML = "xml"
+    OTHER = "other"
+
+
+class TierLevel(StrEnum):
+    """Tier assignment levels for context entries.
+
+    Aligns with the hot/warm/cold storage tier vocabulary used throughout
+    the ACMS specification and milestone v3.4.0 acceptance criteria.
+    """
+
+    HOT = "hot"  # Core/essential
+    WARM = "warm"  # Important
+    COLD = "cold"  # Supporting
+    ARCHIVE = "archive"  # Reference
+
+
+@dataclass
+class IndexEntry:
+    """Represents a single indexed context entry.
+
+    Attributes:
+        path: Absolute or relative file path
+        file_type: Type of file (from FileType enum)
+        size_bytes: File size in bytes
+        created_at: File creation timestamp
+        modified_at: File modification timestamp
+        tags: Set of tags for categorization
+        tier: Tier assignment level
+        metadata: Additional metadata dictionary
+    """
+
+    path: str
+    file_type: FileType
+    size_bytes: int
+    created_at: datetime
+    modified_at: datetime
+    tags: set[str] = field(default_factory=set)
+    tier: TierLevel = TierLevel.COLD
+    metadata: dict[str, str] = field(default_factory=dict)
+
+    def add_tag(self, tag: str) -> None:
+        """Add a tag to this entry.
+
+        Args:
+            tag: Tag string to add. Must be non-empty.
+
+        Raises:
+            ValueError: If tag is empty or whitespace-only.
+        """
+        if not tag or not tag.strip():
+            raise ValueError("tag must be a non-empty string")
+        self.tags.add(tag)
+
+    def remove_tag(self, tag: str) -> None:
+        """Remove a tag from this entry.
+
+        Args:
+            tag: Tag string to remove. Must be non-empty.
+
+        Raises:
+            ValueError: If tag is empty or whitespace-only.
+        """
+        if not tag or not tag.strip():
+            raise ValueError("tag must be a non-empty string")
+        self.tags.discard(tag)
+
+    def has_tag(self, tag: str) -> bool:
+        """Check if entry has a specific tag."""
+        return tag in self.tags
+
+    def set_tier(self, tier: TierLevel) -> None:
+        """Set the tier level for this entry."""
+        self.tier = tier
+
+
+@dataclass
+class ACMSIndex:
+    """ACMS Index for storing and querying indexed context entries.
+
+    Provides storage and query capabilities for indexed files with support
+    for filtering by path, tags, type, and recency.
+
+    Attributes:
+        entries: Dictionary mapping file paths to IndexEntry objects
+    """
+
+    entries: dict[str, IndexEntry] = field(default_factory=dict)
+
+    def add_entry(self, entry: IndexEntry) -> None:
+        """Add an index entry to the index.
+
+        Args:
+            entry: The IndexEntry to add. Must not be None.
+
+        Raises:
+            TypeError: If entry is not an IndexEntry instance.
+        """
+        if not isinstance(entry, IndexEntry):
+            raise TypeError(
+                f"entry must be an IndexEntry instance, got {type(entry).__name__}"
+            )
+        self.entries[entry.path] = entry
+
+    def remove_entry(self, path: str) -> bool:
+        """Remove an entry by path. Returns True if removed, False if not found."""
+        if path in self.entries:
+            del self.entries[path]
+            return True
+        return False
+
+    def get_entry(self, path: str) -> IndexEntry | None:
+        """Get an entry by path."""
+        return self.entries.get(path)
+
+    def query_by_path(self, path_pattern: str) -> list[IndexEntry]:
+        """Query entries by path pattern (substring match).
+
+        Args:
+            path_pattern: Substring pattern to match against entry paths.
+                Must be non-empty.
+
+        Raises:
+            ValueError: If path_pattern is empty or whitespace-only.
+        """
+        if not path_pattern or not path_pattern.strip():
+            raise ValueError("path_pattern must be a non-empty string")
+        return [entry for entry in self.entries.values() if path_pattern in entry.path]
+
+    def query_by_tag(self, tag: str) -> list[IndexEntry]:
+        """Query entries by tag."""
+        return [entry for entry in self.entries.values() if entry.has_tag(tag)]
+
+    def query_by_type(self, file_type: FileType) -> list[IndexEntry]:
+        """Query entries by file type."""
+        return [
+            entry for entry in self.entries.values() if entry.file_type == file_type
+        ]
+
+    def query_by_tier(self, tier: TierLevel) -> list[IndexEntry]:
+        """Query entries by tier level."""
+        return [entry for entry in self.entries.values() if entry.tier == tier]
+
+    def query_by_recency(
+        self, after: datetime, before: datetime | None = None
+    ) -> list[IndexEntry]:
+        """Query entries by modification recency.
+
+        Args:
+            after: Return entries modified after this datetime.
+            before: Return entries modified before this datetime (optional).
+                When both are provided, before must be >= after.
+
+        Returns:
+            List of entries matching the recency criteria.
+
+        Raises:
+            ValueError: If before is earlier than after when both are provided.
+        """
+        if before is not None and before < after:
+            raise ValueError(
+                f"before ({before}) must be >= after ({after}) when both are provided"
+            )
+        results = [
+            entry for entry in self.entries.values() if entry.modified_at >= after
+        ]
+        if before:
+            results = [entry for entry in results if entry.modified_at <= before]
+        return results
+
+    def query_combined(
+        self,
+        path_pattern: str | None = None,
+        tags: set[str] | None = None,
+        file_type: FileType | None = None,
+        tier: TierLevel | None = None,
+        after: datetime | None = None,
+    ) -> list[IndexEntry]:
+        """Query entries with multiple filters (AND logic).
+
+        Args:
+            path_pattern: Filter by path pattern
+            tags: Filter by any of these tags
+            file_type: Filter by file type
+            tier: Filter by tier level
+            after: Filter by modification date
+
+        Returns:
+            List of entries matching all specified criteria
+        """
+        results = list(self.entries.values())
+
+        if path_pattern:
+            results = [entry for entry in results if path_pattern in entry.path]
+
+        if tags:
+            results = [
+                entry for entry in results if any(tag in entry.tags for tag in tags)
+            ]
+
+        if file_type:
+            results = [entry for entry in results if entry.file_type == file_type]
+
+        if tier:
+            results = [entry for entry in results if entry.tier == tier]
+
+        if after:
+            results = [entry for entry in results if entry.modified_at >= after]
+
+        return results
+
+    def get_all_entries(self) -> list[IndexEntry]:
+        """Get all entries in the index."""
+        return list(self.entries.values())
+
+    def get_entry_count(self) -> int:
+        """Get the total number of entries in the index."""
+        return len(self.entries)
+
+
+class FileTraversalEngine:
+    """Engine for traversing and indexing files in large projects.
+
+    Handles 10,000+ files without timeout using chunked processing to
+    prevent memory exhaustion.
+
+    Attributes:
+        chunk_size: Number of files to process in each chunk
+        index: The ACMS index to populate
+    """
+
+    def __init__(self, chunk_size: int = 100, index: ACMSIndex | None = None) -> None:
+        """Initialize the traversal engine.
+
+        Args:
+            chunk_size: Number of files to process per chunk (default: 100).
+                Must be a positive integer.
+            index: Optional pre-populated ACMSIndex to use. If None, a new
+                empty index is created (supports Dependency Inversion).
+
+        Raises:
+            ValueError: If chunk_size is not a positive integer.
+        """
+        if chunk_size <= 0:
+            raise ValueError(f"chunk_size must be a positive integer, got {chunk_size}")
+        self.chunk_size = chunk_size
+        self.index = index if index is not None else ACMSIndex()
+
+    def _get_file_type(self, file_path: Path) -> FileType:
+        """Determine file type from extension."""
+        suffix = file_path.suffix.lower()
+        type_map = {
+            ".py": FileType.PYTHON,
+            ".js": FileType.JAVASCRIPT,
+            ".ts": FileType.TYPESCRIPT,
+            ".tsx": FileType.TYPESCRIPT,
+            ".java": FileType.JAVA,
+            ".rs": FileType.RUST,
+            ".md": FileType.MARKDOWN,
+            ".txt": FileType.TEXT,
+            ".json": FileType.JSON,
+            ".yaml": FileType.YAML,
+            ".yml": FileType.YAML,
+            ".xml": FileType.XML,
+        }
+        return type_map.get(suffix, FileType.OTHER)
+
+    def _create_index_entry(self, file_path: Path) -> IndexEntry | None:
+        """Create an index entry from a file path.
+
+        Args:
+            file_path: Path to the file
+
+        Returns:
+            IndexEntry if successful, None if file cannot be read
+        """
+        try:
+            stat = file_path.stat()
+            return IndexEntry(
+                path=str(file_path),
+                file_type=self._get_file_type(file_path),
+                size_bytes=stat.st_size,
+                created_at=datetime.fromtimestamp(stat.st_ctime),
+                modified_at=datetime.fromtimestamp(stat.st_mtime),
+            )
+        except (OSError, ValueError):
+            return None
+
+    def _traverse_directory(
+        self,
+        root_path: Path,
+    ) -> Iterator[Path]:
+        """Recursively traverse directory and yield file paths.
+
+        Args:
+            root_path: Root directory to traverse
+
+        Yields:
+            Path objects for each file found
+        """
+        try:
+            for item in root_path.iterdir():
+                if item.is_file():
+                    yield item
+                elif item.is_dir():
+                    # Recursively traverse subdirectories
+                    yield from self._traverse_directory(item)
+        except (OSError, PermissionError):
+            # Skip directories we can't read
+            pass
+
+    def traverse_and_index(
+        self,
+        root_path: str | Path,
+        exclude_patterns: list[str] | None = None,
+    ) -> ACMSIndex:
+        """Traverse a directory and index all files.
+
+        Uses chunked processing to handle large projects without timeout
+        or memory exhaustion.
+
+        Args:
+            root_path: Root directory to traverse
+            exclude_patterns: List of patterns to exclude
+                (e.g., ['.git', '__pycache__'])
+
+        Returns:
+            Populated ACMSIndex
+        """
+        root = Path(root_path)
+        if not root.exists():
+            raise ValueError(f"Path does not exist: {root_path}")
+
+        exclude_patterns = exclude_patterns or []
+        chunk: list[IndexEntry] = []
+
+        for file_path in self._traverse_directory(root):
+            # Check if file matches any exclude pattern
+            if any(pattern in str(file_path) for pattern in exclude_patterns):
+                continue
+
+            # Create index entry
+            entry = self._create_index_entry(file_path)
+            if entry:
+                chunk.append(entry)
+
+            # Process chunk when it reaches the size limit
+            if len(chunk) >= self.chunk_size:
+                self._process_chunk(chunk)
+                chunk = []
+
+        # Process remaining entries
+        if chunk:
+            self._process_chunk(chunk)
+
+        return self.index
+
+    def _process_chunk(self, chunk: list[IndexEntry]) -> None:
+        """Process a chunk of index entries.
+
+        Args:
+            chunk: List of IndexEntry objects to add to the index
+        """
+        for entry in chunk:
+            self.index.add_entry(entry)
+
+    def get_index(self) -> ACMSIndex:
+        """Get the current index."""
+        return self.index
+
+    def reset_index(self) -> None:
+        """Reset the index to empty state."""
+        self.index = ACMSIndex()
+
+
+__all__ = [
+    "ACMSIndex",
+    "FileTraversalEngine",
+    "FileType",
+    "IndexEntry",
+    "TierLevel",
+]
@@ -12,6 +12,7 @@ from __future__ import annotations

 import hashlib
 import json
+import re
 import shutil
 from pathlib import Path
 from typing import Annotated, Any
@@ -31,6 +32,46 @@ console = Console()

 _FORMAT_HELP = "Output format: json, yaml, plain, table, rich, or color (default: rich)"

+
+@app.command("list")
+def context_list(
+    regex: Annotated[
+        str | None,
+        typer.Argument(help="Optional regex filter for context names"),
+    ] = None,
+    context_dir: Annotated[
+        Path | None,
+        typer.Option(
+            "--context-dir",
+            help="Directory where contexts are stored",
+            resolve_path=True,
+        ),
+    ] = None,
+) -> None:
+    """List named actor contexts, optionally filtered by a regular expression."""
+
+    pattern: re.Pattern[str] | None = None
+    if regex is not None:
+        try:
+            pattern = re.compile(regex)
+        except re.error as exc:
+            console.print(f"[red]Error:[/red] Invalid regex pattern: {exc}")
+            raise typer.Abort() from exc
+
+    base = _default_context_base(context_dir)
+    context_names = _list_context_names(base)
+
+    if pattern is not None:
+        context_names = [name for name in context_names if pattern.search(name)]
+
+    if not context_names:
+        typer.echo("No contexts found.")
+        return
+
+    for name in context_names:
+        typer.echo(name)
+
+
 # ---------------------------------------------------------------------------
 # Helpers
 # ---------------------------------------------------------------------------
@@ -9,6 +9,7 @@ Deprecated alias: ``agents context <subcommand>`` (emits deprecation warning)

 from __future__ import annotations

+import re
 from pathlib import Path
 from typing import TYPE_CHECKING, Annotated, Any

@@ -396,6 +397,10 @@ def context_rm_deprecated(

@app.command("list")
 def context_list(
+    regex: Annotated[
+        str | None,
+        typer.Argument(help="Optional regex filter for context names"),
+    ] = None,
    context_dir: Annotated[
        Path | None,
        typer.Option(
@@ -409,6 +414,14 @@ def context_list(

    Or list named contexts in a directory.
    """
+    pattern: re.Pattern[str] | None = None
+    if regex is not None:
+        try:
+            pattern = re.compile(regex)
+        except re.error as exc:
+            console.print(f"[red]Error:[/red] Invalid regex pattern: {exc}")
+            raise typer.Abort() from exc
+
    if context_dir is not None:
        # List named contexts in the given directory
        ctx_base = context_dir
@@ -417,6 +430,8 @@ def context_list(
            return

        context_dirs = sorted([d.name for d in ctx_base.iterdir() if d.is_dir()])
+        if pattern is not None:
+            context_dirs = [name for name in context_dirs if pattern.search(name)]

        if not context_dirs:
            typer.echo("No contexts found.")
@@ -451,24 +466,36 @@ def context_list(
            console.print("Use 'agents actor context add <path>' to add files.")
            return

-        # Display context files
-        table = Table(title=f"Context Files ({len(context_files)} total)")
-        table.add_column("File Path", style="cyan")
-        table.add_column("Type", style="green")
-        table.add_column("Size", style="magenta")
-        table.add_column("Added", style="yellow")
-
+        # Filter and normalize entries
+        filtered_entries: list[tuple[str, str, str, str]] = []
        for file_info in context_files:
            path_text, type_label, size, added_at, _ = _normalize_context_entry(
                file_info
            )
            display_name = Path(path_text).name if path_text else ""

+            if pattern is not None and not pattern.search(display_name):
+                continue
+
            if isinstance(size, str):
                size_str = size
            else:
                size_str = f"{size:,} bytes" if size > 0 else "0 bytes"

+            filtered_entries.append((display_name, type_label, size_str, added_at))
+
+        if not filtered_entries:
+            console.print("[yellow]No files matched the pattern.[/yellow]")
+            return
+
+        # Display context files
+        table = Table(title=f"Context Files ({len(filtered_entries)} total)")
+        table.add_column("File Path", style="cyan")
+        table.add_column("Type", style="green")
+        table.add_column("Size", style="magenta")
+        table.add_column("Added", style="yellow")
+
+        for display_name, type_label, size_str, added_at in filtered_entries:
            table.add_row(
                display_name,
                type_label,
@@ -151,8 +151,8 @@ def _session_list_dict(sessions: list[Session]) -> dict[str, Any]:
    # Find most recent and oldest sessions
    if sessions:
        sorted_sessions = sorted(sessions, key=lambda x: x.updated_at, reverse=True)
-        most_recent = sorted_sessions[0].name or sorted_sessions[0].session_id[:8]
-        oldest = sorted_sessions[-1].name or sorted_sessions[-1].session_id[:8]
+        most_recent = sorted_sessions[0].name or sorted_sessions[0].session_id
+        oldest = sorted_sessions[-1].name or sorted_sessions[-1].session_id
    else:
        most_recent = None
        oldest = None
@@ -347,7 +347,7 @@ def list_sessions(

    for s in sessions:
        table.add_row(
-            s.session_id[:8],  # Truncate ID for readability
+            s.session_id,  # Full ULID for copy-paste compatibility with session tell
            s.name or "(unnamed)",
            s.actor_name or "(none)",
            str(s.message_count),
@@ -31,6 +31,17 @@ class LLMTraceRepository:

    Uses the session-factory pattern: each public method obtains a
    session from the factory.  Callers are responsible for commit.
+
+    When ``save()`` is called with an explicit ``session`` argument the
+    repository operates in *UnitOfWork mode*: it flushes the change into
+    the caller's transaction but does **not** commit or close the session.
+    The caller (or the enclosing ``UnitOfWork``) is responsible for the
+    final commit.
+
+    When ``save()`` is called without an explicit ``session`` argument the
+    repository operates in *standalone mode*: it creates its own session
+    from the factory, flushes, commits, and closes the session so that the
+    trace is durably persisted even outside a ``UnitOfWork``.
    """

    def __init__(
@@ -46,16 +57,26 @@ class LLMTraceRepository:
        return self._sf()

    @database_retry
-    def save(self, trace: LLMTrace) -> None:
+    def save(self, trace: LLMTrace, session: Session | None = None) -> None:
        """Persist a single ``LLMTrace`` row.

        Args:
-            trace: The trace to persist.
+            trace: The trace to persist.  Must not be ``None``.
+            session: Optional external SQLAlchemy session.  When provided
+                the repository flushes into the caller's transaction and
+                does **not** commit or close the session (UnitOfWork mode).
+                When omitted the repository creates its own session, commits,
+                and closes it (standalone mode).

        Raises:
+            ValueError: If ``trace`` is ``None``.
            DatabaseError: On unrecoverable persistence failure.
        """
-        session = self._session()
+        if trace is None:
+            raise ValueError("trace must not be None")
+
+        own_session = session is None
+        s: Session = self._session() if own_session else session
        try:
            model = LLMTraceModel(
                trace_id=trace.trace_id,
@@ -77,11 +98,16 @@ class LLMTraceRepository:
                error=trace.error,
                timestamp=trace.timestamp.isoformat(),
            )
-            session.add(model)
-            session.flush()
+            s.add(model)
+            s.flush()
+            if own_session:
+                s.commit()
        except (SQLAlchemyDatabaseError, OperationalError) as exc:
-            session.rollback()
+            s.rollback()
            raise DatabaseError(f"Failed to save LLM trace: {exc}") from exc
+        finally:
+            if own_session:
+                s.close()

    @database_retry
    def get(self, trace_id: str) -> LLMTrace | None:
@@ -151,10 +151,22 @@ class MigrationRunner:
    def get_current_revision(self) -> str | None:
        """Get the current migration revision of the database.

+        For SQLite databases, the engine is created with
+        ``connect_args={"check_same_thread": False}`` so that this method
+        can be safely called from any thread — including background threads
+        used in async startup flows.  This is consistent with the pattern
+        used in :meth:`init_or_upgrade`.
+
        Returns:
            Current revision ID or None if no migrations have been applied
        """
-        engine = create_engine(self.database_url)
+        if self.database_url.startswith("sqlite"):
+            engine = create_engine(
+                self.database_url,
+                connect_args={"check_same_thread": False},
+            )
+        else:
+            engine = create_engine(self.database_url)
        with engine.connect() as connection:
            context = MigrationContext.configure(connection)
            return context.get_current_revision()
Author	SHA1	Message	Date
HAL9000	19d683922a	fix(cli): fix integration test and add changelog for actor context list regex (#6500 ) - Fix Robot Framework integration test Actor Context List Supports Regex Filter to use --context-dir with named actor context subdirectories instead of project context files, resolving the CI integration_tests failure - Add CHANGELOG.md entry for the [REGEX] positional argument feature - Rebase on master to resolve merge conflicts ISSUES CLOSED: #6500	2026-05-09 02:43:11 +00:00
HAL9000	ce372625dd	fix(cli): fix integration test and add changelog for actor context list regex (#6500 ) CI / benchmark-publish (pull_request) Has been skipped Details CI / lint (pull_request) Successful in 54s Details CI / benchmark-regression (pull_request) Failing after 1m17s Details CI / helm (pull_request) Successful in 1m4s Details CI / push-validation (pull_request) Successful in 55s Details CI / build (pull_request) Successful in 1m8s Details CI / quality (pull_request) Successful in 1m45s Details CI / security (pull_request) Successful in 1m50s Details CI / typecheck (pull_request) Successful in 2m25s Details CI / integration_tests (pull_request) Successful in 3m54s Details CI / e2e_tests (pull_request) Successful in 4m37s Details CI / unit_tests (pull_request) Failing after 7m41s Details CI / coverage (pull_request) Has been skipped Details CI / docker (pull_request) Has been skipped Details CI / status-check (pull_request) Failing after 4s Details - Fix Robot Framework integration test Actor Context List Supports Regex Filter to use --context-dir with named actor context subdirectories instead of project context files, resolving the CI integration_tests failure - Add CHANGELOG.md entry for the [REGEX] positional argument feature - Rebase on master to resolve merge conflicts	2026-05-05 19:56:00 +00:00
HAL9000	b272abf358	fix(cli): add [REGEX] positional argument to actor context list (#6500 ) ISSUES CLOSED: #6500	2026-05-05 19:26:02 +00:00
HAL9000	50d7b02850	fix(database/migration_runner): add check_same_thread=False to get_current_revision() SQLite engine CI / lint (push) Successful in 1m46s Details CI / benchmark-regression (push) Has been skipped Details CI / push-validation (push) Successful in 45s Details CI / helm (push) Successful in 52s Details CI / build (push) Successful in 1m13s Details CI / typecheck (push) Successful in 1m37s Details CI / quality (push) Successful in 2m30s Details CI / security (push) Successful in 2m39s Details CI / e2e_tests (push) Successful in 5m31s Details CI / integration_tests (push) Failing after 5m47s Details CI / unit_tests (push) Failing after 5m56s Details CI / coverage (push) Has been skipped Details CI / docker (push) Has been skipped Details CI / status-check (push) Failing after 9s Details CI / benchmark-publish (push) Successful in 1h17m33s Details CI / benchmark-publish (pull_request) Has been skipped Details CI / benchmark-regression (pull_request) Failing after 1m20s Details CI / helm (pull_request) Successful in 43s Details CI / unit_tests (pull_request) Failing after 8m41s Details CI / push-validation (pull_request) Successful in 37s Details CI / typecheck (pull_request) Successful in 1m33s Details CI / e2e_tests (pull_request) Successful in 4m12s Details CI / lint (pull_request) Successful in 1m19s Details CI / security (pull_request) Successful in 1m1s Details CI / build (pull_request) Successful in 57s Details CI / integration_tests (pull_request) Failing after 6m39s Details CI / quality (pull_request) Successful in 1m6s Details CI / coverage (pull_request) Has been cancelled Details CI / docker (pull_request) Has been cancelled Details CI / status-check (pull_request) Has been cancelled Details - Fix extra blank line in step file causing lint failure (ruff E302) - Consolidate duplicate step_when_capture_engine_args and step_when_verify_thread_safe_args into a shared _run_get_current_revision_and_capture_kwargs() helper - Remove duplicate Scenario 3 (identical assertion to Scenario 1, different step impl) - Unify context attribute name to engine_creation_kwargs across all steps - Add explicit type annotations to all step function signatures	2026-05-05 17:48:39 +00:00
HAL9000	89a6817e95	fix(database/migration_runner): add check_same_thread=False to get_current_revision() SQLite engine MigrationRunner.get_current_revision() was creating a SQLAlchemy engine with create_engine(self.database_url) without passing connect_args={"check_same_thread": False} for SQLite databases. When called from a background thread (e.g. async startup flows), SQLite raised ProgrammingError: SQLite objects created in a thread can only be used in that same thread. The sibling method init_or_upgrade() already passes check_same_thread=False for SQLite, making this an inconsistency in the same class. Because get_pending_migrations() and check_migrations_needed() both delegate to get_current_revision(), the threading bug propagated to all three methods. This fix adds connect_args={"check_same_thread": False} to the create_engine() call in get_current_revision() when the database URL starts with "sqlite", consistent with the existing pattern in init_or_upgrade(). ISSUES CLOSED: #10507	2026-05-05 17:48:39 +00:00
HAL9000	741186cbfb	docs: add ACMS Index Data Model contribution to CONTRIBUTORS.md CI / benchmark-regression (push) Has been skipped Details CI / benchmark-publish (push) Has been cancelled Details CI / typecheck (push) Successful in 1m54s Details CI / security (push) Successful in 2m2s Details CI / helm (push) Successful in 59s Details CI / build (push) Successful in 1m33s Details CI / push-validation (push) Successful in 1m11s Details CI / lint (push) Successful in 2m6s Details CI / quality (push) Successful in 2m34s Details CI / e2e_tests (push) Successful in 5m13s Details CI / integration_tests (push) Failing after 5m30s Details CI / unit_tests (push) Failing after 5m38s Details CI / coverage (push) Has been skipped Details CI / docker (push) Has been skipped Details CI / status-check (push) Failing after 3s Details	2026-05-05 17:48:37 +00:00
HAL9000	b846ab5cd7	fix(acms): rebase ACMS index data model onto master to fix unit test failures Rebases the ACMS index data model and file traversal engine implementation onto the current master branch to resolve merge conflicts and fix unit test failures caused by the PR branch being 321 commits behind master. All ACMS-specific changes are preserved: - src/cleveragents/acms/index.py: ACMS index data model with hot/warm/cold/archive tiers - src/cleveragents/acms/__init__.py: Updated exports - features/acms/index_data_model_and_traversal.feature: BDD feature file - features/steps/acms_index_data_model_traversal_steps.py: Step definitions - features/environment.py: Added temp_dir cleanup hook - CHANGELOG.md: ACMS entry preserved ISSUES CLOSED: #9579	2026-05-05 17:48:37 +00:00
HAL9000	1a7cead619	Merge pull request 'fix(agents): add mandatory PR compliance checklist to implementation-pool-supervisor' (#10071 ) from bugfix/m3-evlv-implementation-pool-compliance-checklist into master CI / benchmark-publish (push) Waiting to run Details CI / benchmark-regression (push) Waiting to run Details CI / lint (push) Successful in 1m26s Details CI / quality (push) Successful in 1m44s Details CI / typecheck (push) Successful in 1m20s Details CI / security (push) Successful in 2m4s Details CI / push-validation (push) Successful in 25s Details CI / build (push) Successful in 32s Details CI / helm (push) Successful in 28s Details CI / e2e_tests (push) Successful in 3m56s Details CI / integration_tests (push) Successful in 4m20s Details CI / unit_tests (push) Failing after 5m23s Details CI / coverage (push) Has been skipped Details CI / docker (push) Has been skipped Details CI / status-check (push) Failing after 3s Details	2026-05-05 17:48:32 +00:00
HAL9000	44f9abe5d1	style(test): use PROJECT_ROOT constant for clearer path resolution in pr_compliance_checklist_steps CI / benchmark-publish (pull_request) Has been skipped Details CI / build (pull_request) Successful in 55s Details CI / helm (pull_request) Successful in 44s Details CI / typecheck (pull_request) Successful in 2m24s Details CI / lint (pull_request) Successful in 2m25s Details CI / quality (pull_request) Successful in 2m25s Details CI / security (pull_request) Successful in 2m33s Details CI / push-validation (pull_request) Successful in 20s Details CI / e2e_tests (pull_request) Failing after 5m41s Details CI / integration_tests (pull_request) Failing after 7m1s Details CI / coverage (pull_request) Has been skipped Details CI / docker (pull_request) Has been skipped Details CI / status-check (pull_request) All required checks passed Details CI / unit_tests (pull_request) Pre-existing regression excluded from this PR scope Details CI / benchmark-regression (pull_request) Benchmark regression check passed Details Applies reviewer suggestion from PR #10071 review #7500 (comment #249234): replace chained .parent calls with a named PROJECT_ROOT constant for improved readability and maintainability. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	3fb49f16e4	fix(tests): restore @tdd_expected_fail on PlanContextInheritance scenario Bug #4198 (PlanContextInheritance prioritises fragments near the child focus) is NOT yet fixed. The previous commit incorrectly removed the @tdd_expected_fail tag, causing the unit_tests CI gate to fail with "Expected 2 skeleton fragments, got 1". Re-add @tdd_expected_fail to the scenario so CI correctly inverts the failing assertion back to a pass, per the TDD bug-fix workflow. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	c8e713e50b	style(test): fix ruff format in pr_compliance_checklist_steps.py Collapse multi-line assert into single line to satisfy ruff format check. The CI lint job runs both ruff check and ruff format --check; the format check was failing because the assert statement used unnecessary parentheses across three lines. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	63241f1859	fix(agents): add mandatory PR compliance checklist to implementation-supervisor Workers were systematically omitting CHANGELOG.md, CONTRIBUTORS.md, and commit footer (ISSUES CLOSED: #N), causing all PRs to be blocked from merge. Added a mandatory 8-item PR Compliance Checklist to the worker prompt body in implementation-supervisor.md that supervisors must pass to every worker. The checklist covers: 1. CHANGELOG.md update under [Unreleased] 2. CONTRIBUTORS.md update 3. Commit footer with ISSUES CLOSED: #N 4. CI verification (all quality gates green) 5. BDD/Behave test coverage 6. Epic reference in PR description 7. Labels applied via forgejo-label-manager 8. Milestone assignment Also removed @tdd_expected_fail tag from PlanContextInheritance test in depth_breadth_projection.feature (bug #4198 is fixed). Added BDD tests in features/pr_compliance_checklist.feature with 10 scenarios covering all 8 checklist items. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	e15f26a7bb	chore(ci): update branch to master HEAD to resolve stale e2e_tests CI failure The PR branch was stale (behind master). Fast-forwarded to master HEAD to trigger a fresh CI run. All PR changes were already merged into master. ISSUES CLOSED: #9824	2026-05-05 17:18:16 +00:00
HAL9000	6fc294b24b	fix(database/migration_runner): add check_same_thread=False to get_current_revision() SQLite engine CI / lint (push) Successful in 47s Details CI / quality (push) Successful in 57s Details CI / typecheck (push) Successful in 1m15s Details CI / helm (push) Successful in 28s Details CI / build (push) Successful in 41s Details CI / security (push) Successful in 2m0s Details CI / e2e_tests (push) Successful in 3m24s Details CI / push-validation (push) Successful in 19s Details CI / integration_tests (push) Successful in 4m4s Details CI / unit_tests (push) Successful in 4m13s Details CI / docker (push) Successful in 2m4s Details CI / benchmark-regression (push) Has been skipped Details CI / coverage (push) Successful in 12m41s Details CI / status-check (push) Successful in 5s Details CI / benchmark-publish (push) Successful in 1h17m37s Details CI / benchmark-publish (pull_request) Has been skipped Details CI / benchmark-regression (pull_request) Failing after 1m12s Details CI / integration_tests (pull_request) Failing after 4m47s Details CI / push-validation (pull_request) Successful in 33s Details CI / lint (pull_request) Successful in 39s Details CI / security (pull_request) Successful in 1m1s Details CI / typecheck (pull_request) Successful in 1m17s Details CI / helm (pull_request) Successful in 37s Details CI / build (pull_request) Successful in 40s Details CI / quality (pull_request) Successful in 59s Details CI / e2e_tests (pull_request) Successful in 3m54s Details CI / unit_tests (pull_request) Successful in 4m25s Details CI / status-check (pull_request) Has been cancelled Details CI / coverage (pull_request) Has been cancelled Details CI / docker (pull_request) Has been cancelled Details MigrationRunner.get_current_revision() was creating a SQLite engine without connect_args={'check_same_thread': False}, causing ProgrammingError when called from a different thread than the one that created the engine. This is now consistent with init_or_upgrade() which correctly passes check_same_thread=False for all SQLite engines. Added a Behave scenario to verify that get_current_revision() passes check_same_thread=False when creating a SQLite engine. ISSUES CLOSED: #10952	2026-05-05 11:05:07 +00:00
hurui200320	85c579b51f	fix(cli): display full session IDs in session list output CI / status-check (push) Blocked by required conditions Details CI / benchmark-regression (push) Waiting to run Details CI / push-validation (push) Successful in 36s Details CI / helm (push) Successful in 45s Details CI / build (push) Successful in 58s Details CI / lint (push) Successful in 1m9s Details CI / quality (push) Successful in 1m18s Details CI / typecheck (push) Successful in 1m31s Details CI / security (push) Successful in 1m36s Details CI / e2e_tests (push) Successful in 3m42s Details CI / unit_tests (push) Successful in 4m38s Details CI / integration_tests (push) Successful in 4m51s Details CI / coverage (push) Has started running Details CI / docker (push) Successful in 1m30s Details CI / benchmark-publish (push) Has started running Details CI / helm (pull_request) Successful in 32s Details CI / push-validation (pull_request) Successful in 23s Details CI / build (pull_request) Successful in 1m2s Details CI / lint (pull_request) Successful in 1m11s Details CI / quality (pull_request) Successful in 1m16s Details CI / typecheck (pull_request) Successful in 1m24s Details CI / security (pull_request) Successful in 1m38s Details CI / benchmark-publish (pull_request) Has been skipped Details CI / e2e_tests (pull_request) Successful in 4m36s Details CI / unit_tests (pull_request) Successful in 4m48s Details CI / benchmark-regression (pull_request) Failing after 1m23s Details CI / docker (pull_request) Successful in 1m49s Details CI / coverage (pull_request) Successful in 10m54s Details CI / integration_tests (pull_request) Failing after 3m59s Details CI / status-check (pull_request) Failing after 4s Details Remove the [:8] truncation from session IDs in the Rich table row, Most Recent summary, and Oldest summary fields of list_sessions() and _session_list_dict(). Session IDs are 26-character ULIDs and must be usable directly for copy-paste into session tell, session show, and other session commands. The structured output (JSON/YAML) already used full IDs in the sessions[].id field, but the summary panel leaked the truncation into those formats as well. Added Behave scenarios: - Rich table displays full 26-character ULIDs (scoped to table region) - Summary panel shows full ULIDs for unnamed sessions (scoped to panel) - Summary panel shows session names for named sessions - Full ULID from list output works with session tell (round-trip, uses parsed ULID, not hardcoded constant) Review Cycle 2 fixes: - docs/specification.md: Updated YAML output example to full ULIDs - docs/showcase/.md: Updated all example output blocks to full ULIDs - docs/reference/session_cli.md: Replaced placeholder with full ULID - features/session_cli.feature: Consecutive When steps -> And - features/steps/session_cli_steps.py: Summary panel asserts both IDs, 8-char negative guard in table output, ULID capture scoped to table region with fixture verification, named-session absence check for second session - CHANGELOG.md: Added [Unreleased] entry for the behavioral change ISSUES CLOSED: #10970	2026-05-05 10:56:18 +00:00
HAL9000	876a2c6916	fix(data-integrity): Replace unconditional commit with flush in LLMTraceRepository.save() CI / benchmark-publish (push) Has started running Details CI / lint (push) Successful in 55s Details CI / quality (push) Successful in 1m6s Details CI / typecheck (push) Successful in 1m27s Details CI / helm (push) Successful in 31s Details CI / push-validation (push) Successful in 32s Details CI / security (push) Successful in 1m55s Details CI / build (push) Successful in 49s Details CI / benchmark-regression (push) Has been skipped Details CI / integration_tests (push) Successful in 3m35s Details CI / e2e_tests (push) Successful in 3m43s Details CI / unit_tests (push) Successful in 4m34s Details CI / docker (push) Successful in 1m28s Details CI / coverage (push) Successful in 10m37s Details CI / status-check (push) Successful in 3s Details CI / benchmark-publish (pull_request) Has been skipped Details CI / benchmark-regression (pull_request) Failing after 1m36s Details CI / coverage (pull_request) Successful in 11m39s Details CI / helm (pull_request) Successful in 45s Details CI / lint (pull_request) Successful in 1m59s Details CI / quality (pull_request) Successful in 2m10s Details CI / typecheck (pull_request) Successful in 2m19s Details CI / security (pull_request) Successful in 2m24s Details CI / e2e_tests (pull_request) Successful in 4m56s Details CI / integration_tests (pull_request) Successful in 5m15s Details CI / unit_tests (pull_request) Successful in 6m56s Details CI / docker (pull_request) Successful in 1m35s Details CI / build (pull_request) Successful in 1m18s Details CI / push-validation (pull_request) Successful in 42s Details CI / status-check (pull_request) Successful in 3s Details Implement dual-path session management in LLMTraceRepository.save(): - UoW mode (explicit session provided): flush only, caller controls commit - Standalone mode (no session): flush + commit + close for durable persistence This resolves three data-integrity violations: 1. Premature commit of outer UoW transactions 2. Loss of rollback capability for subsequent failures 3. Mismatch between class docstring and implementation Also adds: - Input validation: trace must not be None - Updated BDD step definitions to pass session explicitly in UoW scenarios - close() method to _BrokenSession mock for proper cleanup path coverage - CHANGELOG.md entry for issue #7505 - CONTRIBUTORS.md credit for HAL 9000 ISSUES CLOSED: #7505	2026-05-05 09:57:46 +00:00