Merge pull request #29 from PromptExecution/codex-issue-21-workbook-contract

feat: honor the CPA workbook export contract

Author: elasticdotventures

AGENTS.md

@@ -245,6 +245,10 @@ Treat this as a standing operational gate, not a one-time migration task.
   - The published MCP surface now lives in `crates/ledgerr-mcp/src/contract.rs`; treat it as the only source of truth for parser shapes, generated JSON Schema, and checked-in operator docs/examples.
   - Regenerate `docs/mcp-capability-contract.md`, `docs/agent-mcp-runbook.md`, and `scripts/mcp_cli_demo.sh` via `cargo run -p xtask-mcpb -- generate-mcp-artifacts` after changing the published MCP surface.
   - Drift between `contract.rs` and those generated artifacts is a test failure, not a documentation chore.
+- 2026-04-17: CPA workbook export is now explicitly projection-only.
+  - Treat `ledger_core::workbook::REQUIRED_SHEETS` as the canonical base workbook contract for export paths.
+  - `export_cpa_workbook` should rebuild the full workbook from canonical service state on each export, including `META.config`, `ACCT.registry`, schedule sheets, flag sheets, transaction sheets, and `AUDIT.log`.
+  - Tests should assert representative workbook contents, not just that a file was written.
 
 
 

crates/ledgerr-mcp/src/contract.rs

@@ -157,9 +157,8 @@ impl JsonSchema for PluginInfoSubcommand {
     fn json_schema(_gen: &mut schemars::gen::SchemaGenerator) -> Schema {
         Schema::Object(SchemaObject {
             instance_type: Some(SingleOrVec::Single(Box::new(InstanceType::String))),
-            enum_values: Some(vec![json!("check"), json!("upgrade"), json!("cleanup")]),
             metadata: Some(Box::new(Metadata {
-                description: Some("Recognized values are check, upgrade, and cleanup. Unknown strings intentionally fall through to the default check behavior.".to_string()),
+                description: Some("Subcommand string. Known values: check, upgrade, cleanup. Unknown strings fall through to the default check behavior.".to_string()),
                 ..Metadata::default()
             })),
             ..SchemaObject::default()
@@ -599,8 +598,7 @@ Expected blocked outcomes:\n\n\
 }
 
 pub fn generated_mcp_cli_demo_script() -> String {
-    format!(
-        "#!/usr/bin/env bash\nset -euo pipefail\n\n\
+    "#!/usr/bin/env bash\nset -euo pipefail\n\n\
 JOURNAL_PATH=\"${{JOURNAL_PATH:-/tmp/demo.beancount}}\"\n\
 WORKBOOK_PATH=\"${{WORKBOOK_PATH:-/tmp/demo.xlsx}}\"\n\
 SOURCE_REF=\"${{SOURCE_REF:-wf-2023-01.rkyv}}\"\n\n\
@@ -619,7 +617,7 @@ cat <<'EOF'\n\
 {{\"jsonrpc\":\"2.0\",\"id\":3,\"method\":\"tools/call\",\"params\":{{\"name\":\"ledgerr_reconciliation\",\"arguments\":{{\"action\":\"commit\",\"source_total\":\"100.00\",\"extracted_total\":\"95.00\",\"posting_amounts\":[\"-95.00\",\"95.00\"]}}}}}}\n\
 {{\"jsonrpc\":\"2.0\",\"id\":4,\"method\":\"tools/call\",\"params\":{{\"name\":\"ledgerr_audit\",\"arguments\":{{\"action\":\"event_history\",\"time_start\":\"2026-12-31\",\"time_end\":\"2026-01-01\"}}}}}}\n\
 EOF\n"
-    )
+        .to_string()
 }
 
 fn documents_ingest_pdf_example() -> Value {