Skip to content

Persist Schema & Spec Sync state across runs - #9262

Merged
lpcox merged 4 commits into
mainfrom
copilot/aw-debug-schema-spec-sync
Sep 30, 2026
Merged

lpcox merged 4 commits into
mainfrom
copilot/aw-debug-schema-spec-sync

Conversation

Copilot AI commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

The workflow’s date-scoped cache could not restore its watermark on the next weekday run, triggering cache_memory_miss despite a supported cold-start fallback.

  • Durable state: Move the watermark to a dedicated repo-memory branch.
  • Cold starts: Treat a missing watermark as expected; scan the last seven days without reporting missing data.
repo-memory:
  branch-name: memory/schema-sync

Copilot AI linked an issue Sep 30, 2026 that may be closed by this pull request
Copilot AI and others added 2 commits September 30, 2026 15:32
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix missing data in Schema & Spec Sync workflow Persist Schema & Spec Sync state across runs Sep 30, 2026
Copilot AI requested a review from lpcox September 30, 2026 15:42
@lpcox
lpcox marked this pull request as ready for review September 30, 2026 15:46
Copilot AI balanced review requested due to automatic review settings September 30, 2026 15:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

The source, generated workflow, and regression coverage consistently implement durable watermark persistence.

Review effort: Balanced
Findings: None

What changed in this PR

Moves Schema & Spec Sync’s watermark to durable repo memory, resolving #9258.

Changes:

  • Replaces date-scoped cache memory with memory/schema-sync.
  • Treats missing state as an expected seven-day cold start.
  • Adds regression tests and regenerates the workflow lock.
File Description
.github/​workflows/​schema-sync.md Configures durable state and cold-start behavior.
.github/​workflows/​schema-sync.lock.yml Adds generated repo-memory clone/push jobs.
scripts/​ci/​schema-sync-workflow.test.ts Verifies source and lock configuration.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@copilot Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Coverage Regression Detected

This PR decreases test coverage. Please add tests to maintain coverage levels.

Overall Coverage

Metric Base PR Delta
Lines 92.71% 92.71% ➡️ +0.00%
Statements 91.22% 91.22% ➡️ +0.00%
Functions 89.30% 89.30% ➡️ +0.00%
Branches 84.29% 84.28% 📉 -0.01%
📁 Per-file Coverage Changes (2 files)
File Lines (Before → After) Statements (Before → After)
src/nvx/one-shot-adapter.ts 83.5% → 82.9% (-0.60%) 80.3% → 79.8% (-0.56%)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@lpcox

lpcox commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

@copilot merge main

…ma-spec-sync

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

Copilot AI commented Sep 30, 2026

Copy link
Copy Markdown
Contributor Author

@copilot merge main

Merged main into the branch in merge commit 7986042 (two parents; no conflicts). The targeted Jest test could not run because dependencies are not installed (jest: not found).

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 92.70% 92.71% ➡️ +0.01%
Statements 91.21% 91.22% 📈 +0.01%
Functions 89.30% 89.30% ➡️ +0.00%
Branches 84.27% 84.28% 📈 +0.01%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@lpcox
lpcox merged commit fae8d6f into main Sep 30, 2026
25 of 26 checks passed
@lpcox
lpcox deleted the copilot/aw-debug-schema-spec-sync branch September 30, 2026 17:36
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.

[aw] Schema & Spec Sync is missing required data

3 participants