Skip to content

[codex] Add zellij output fixture coverage - #45

Merged
dapi merged 1 commit into
mainfrom
feature/issue-24-ft-012-zellij-json-fixture-tests
Jul 7, 2026
Merged

[codex] Add zellij output fixture coverage#45
dapi merged 1 commit into
mainfrom
feature/issue-24-ft-012-zellij-json-fixture-tests

Conversation

@dapi

@dapi dapi commented Jul 7, 2026

Copy link
Copy Markdown
Owner

Summary

Adds deterministic fixture coverage for FT-012 so zellij read-path parsing is protected before live detect/create work depends on it.

Changes

  • Added internal/zellij parser types for supported zellij session-list output and list-panes --json output.
  • Added synthetic valid fixtures for list-sessions --short --no-formatting and list-panes --json --all-style pane data.
  • Added negative fixtures for formatted session output, invalid pane JSON, trailing data, missing required pane fields, and invalid pane IDs.
  • Indexed memory-bank/product/execution-order.md from the product README so the documented memory-bank audit passes.

Impact

This does not add live zellij integration, Codex classification, registry writes, or CLI behavior. It establishes a parser/fixture contract for downstream adapter and detect features.

Validation

  • mise exec -- go test ./...
  • mise exec -- go vet ./...
  • mise exec -- go test -race ./...
  • mise exec -- go build -o /tmp/zelma-ft-012 ./cmd/zelma
  • python3 scripts/check_memory_bank_index.py
  • git diff --check
  • rg -n "zelima|Zelima" . plus filtered check excluding known documentation examples

Closes #24.

@dapi
dapi force-pushed the feature/issue-24-ft-012-zellij-json-fixture-tests branch from 4d5da89 to 6534058 Compare July 7, 2026 23:02
@dapi
dapi marked this pull request as ready for review July 7, 2026 23:03
@dapi
dapi force-pushed the feature/issue-24-ft-012-zellij-json-fixture-tests branch from 6534058 to 451405d Compare July 7, 2026 23:09
@dapi
dapi force-pushed the feature/issue-24-ft-012-zellij-json-fixture-tests branch from 451405d to 00dcaad Compare July 7, 2026 23:30
@dapi
dapi merged commit 1fe4401 into main Jul 7, 2026
1 check passed
@dapi
dapi deleted the feature/issue-24-ft-012-zellij-json-fixture-tests branch July 7, 2026 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FT-012: Zellij JSON Fixture Tests

1 participant