Skip to content

Restore files missing from the patch set: agent skills, .gitignore, CI workflow - #1

Merged
sumitshatwara merged 1 commit into
mainfrom
devin/1789030293-restore-missing-files
Sep 10, 2026
Merged

sumitshatwara merged 1 commit into
mainfrom
devin/1789030293-restore-missing-files

Conversation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

Summary

Compared every file in patches 12 and 15 against this repo (mapping the original nm-ccm-migration/<pkg>/ layout onto src/<pkg>/). Everything landed except three groups, which this PR restores, adapted to the root/src layout and the anonymized naming already used here:

  • .agents/skills/ (all six): assess-form-corpus, convert-xdp-form, prepare-designer-handoff, review-rulebook-change, triage-designer-errors, validate-pdf-output. AGENTS.md, README and every playbook already reference them. Paths rewritten fixtures/ -> src/fixtures/, decisions/ -> src/decisions/, etc.; NM_FIXTURE_ROOT -> CCM_FIXTURE_ROOT (matching src/tools); customer/partner names replaced as in the rest of the repo.
  • .gitignore: src/fixtures/raw/* (hard rule 6 assumed this was already ignored), import_logs/composition_logs *.json, plus out/, caches, *.egg-info/, .venv/.
  • .github/workflows/ccm-migration.yml: same verify job as the original (ruff check, rulebook/dictionary lint, pytest, tools.report, conditional tools.pipeline when vars.CCM_FIXTURE_ROOT is set), without the subdirectory working-directory/path filters, plus ruff format --check.
  • README layout block lists all six skills instead of three.

The two knowledge/nm-*.md files from the patches exist here as knowledge/customer-*.md; not duplicated. Raw customer artifacts are not in the patches and are not added.

Verified locally: ruff check ., ruff format --check ., pytest -q (90 passed, 7 skipped without raw artifacts), rulebook/dictionary imports and tools.report all succeed.

Link to Devin session: https://partner-workshops.devinenterprise.com/sessions/060b9626e9ad430186f22f341a34278b
Open in Devin Desktop: https://partner-workshops.devinenterprise.com/desktop/session/060b9626e9ad430186f22f341a34278b?variant=devin
Requested by: @sumitshatwara

@devin-ai-integration

Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@sumitshatwara
sumitshatwara merged commit 0c8a882 into main Sep 10, 2026
1 check passed
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