---
id: TASK-32
title: >-
  Add config schema tests for zephyr workspace keys
  (workspace/base/url/manifest)
status: Done
assignee:
  - codex
created_date: '2026-03-04 22:17'
updated_date: '2026-03-04 22:22'
labels: []
dependencies: []
references:
  - src/config/schema.rs
---

## Description

<!-- SECTION:DESCRIPTION:BEGIN -->
The zephyr config table now includes workspace/base/url/manifest keys. Add unit tests that verify these keys deserialize correctly and overlay logic handles precedence for url/manifest (and existing workspace/base) so resolution uses expected values.
<!-- SECTION:DESCRIPTION:END -->

## Acceptance Criteria
<!-- AC:BEGIN -->
- [x] #1 Add schema unit tests that parse a TOML snippet containing `[zephyr] workspace/base/url/manifest` and assert the resulting `ZephyrCfg` fields are populated as expected.
- [x] #2 Add overlay tests that verify RHS values for `url` and `manifest` override LHS when provided (and do not override when RHS is None), consistent with existing overlay behavior.
- [x] #3 Tests are placed in `src/config/schema.rs` and pass under `just check`.
- [x] #4 If overlay behavior differs from intended precedence, update the test expectations to match intended behavior or document the discrepancy in a test comment.
<!-- AC:END -->

## Implementation Plan

<!-- SECTION:PLAN:BEGIN -->
1) Add a TOML deserialization test in src/config/schema.rs for [zephyr] workspace/base/url/manifest and assert ZephyrCfg fields.
2) Add overlay tests for url/manifest precedence (RHS overrides LHS when set; RHS None does not override).
3) If overlay behavior differs from intended precedence, document in a test comment and align assertions with intended behavior.
4) Run just check to confirm.
<!-- SECTION:PLAN:END -->

## Implementation Notes

<!-- SECTION:NOTES:BEGIN -->
Added zephyr config deserialization test, url overlay tests, and a manifest overlay behavior test with a note about current precedence. Ran `just check -- -D warnings` (passes).
<!-- SECTION:NOTES:END -->

## Final Summary

<!-- SECTION:FINAL_SUMMARY:BEGIN -->
Added zephyr config schema tests for workspace/base/url/manifest deserialization and overlay behavior, including a note documenting current manifest precedence. Tests: `just check -- -D warnings`.
<!-- SECTION:FINAL_SUMMARY:END -->
