Skip to content

Extract project config application - #42

Merged
sabino merged 1 commit into
mainfrom
m1a-config-application
Jun 16, 2026
Merged

Extract project config application#42
sabino merged 1 commit into
mainfrom
m1a-config-application

Conversation

@sabino

@sabino sabino commented Jun 16, 2026

Copy link
Copy Markdown
Owner

Summary

  • move project model path config and seed docs application from src/project.zig into src/project/config.zig
  • keep src/project.zig as the orchestration facade via module aliases
  • add native Zig coverage for config path matching, depth precedence, package restriction, inline materialization precedence, and seed docs targeting
  • update PLAN.md ownership notes for config application

Validation

  • zig fmt --check src/project.zig src/project/config.zig
  • git diff --check
  • changed-file private/secret/path scan: no matches
  • timeout 180s zig test src/project/config.zig --cache-dir /tmp/dxt-zig-cache --global-cache-dir /tmp/dxt-zig-global-cache (16 passed)
  • timeout 180s zig build --cache-dir /tmp/dxt-zig-cache --global-cache-dir /tmp/dxt-zig-global-cache
  • timeout 240s zig build test --cache-dir /tmp/dxt-zig-cache --global-cache-dir /tmp/dxt-zig-global-cache
  • timeout 240s zig build -Doptimize=ReleaseSafe --cache-dir /tmp/dxt-zig-cache --global-cache-dir /tmp/dxt-zig-global-cache
  • python scripts/check_runtime_boundary.py && python scripts/check_public_safety.py
  • pytest -q tests/test_cli.py (48 passed)
  • pytest -q (52 passed)

Agent review

  • subagent explorer recommended this as the next smallest M1A extraction
  • codex exec rejected the typed-sort alternative as a worse module boundary
  • codex exec diff review: No blocking findings.

@sabino
sabino merged commit f05a8ff into main Jun 16, 2026
2 checks passed
@sabino
sabino deleted the m1a-config-application branch June 16, 2026 12:14
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.

1 participant