Skip to content

Catalog the 14 backlog repos; add the eda type and releaseTrigger none - #247

Merged
ptr727 merged 6 commits into
developfrom
feature/catalog-backlog-repos
Jul 6, 2026
Merged

Catalog the 14 backlog repos; add the eda type and releaseTrigger none#247
ptr727 merged 6 commits into
developfrom
feature/catalog-backlog-repos

Conversation

@ptr727

@ptr727ptr727 commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Resolves the classification backlog (22 cataloged, 0 backlog). Per your decisions:

  1. New `eda` type (KiCadLibrary, DevKitCIoT, EspDinIoT) - github-release data zip, EDA-specific CI validation, deterministic build-time artifact/version injection.
  2. `releaseTrigger: none` for repos with no release automation (WIP or file-history-only).
  3. Non-conformant branch states flagged (MediaTools first-branch, HomeAssistant-Config master, PhotoCleaner/TelemetryImporter develop-only, KiCadLibrary/EspDinIoT/HolidayLights stale/empty main) - recorded, flagged for correction, never permanent.
  4. All 14 flipped to `cataloged` with their (possibly deviated) state + driftNotes.
  5. Config repos (Vantage-Config, HomeAutomation, HolidayLights) in scope; TelemetryImporter noted abandoned; .github noted candidate for removal.

MediaTools static NUGET_API_KEY (forbidden) recorded in driftNotes, publish[] left empty pending OIDC correction. validate.py green.

@ptr727@claude
Catalog the 14 backlog repos; add the eda type and releaseTrigger none
48f75e0
Classify and catalog every backlog repo (status -> cataloged with types, fields, and driftNotes). Per maintainer decisions: add a first-class 'eda' type for KiCad/PCB repos (github-release data zip, EDA-specific CI validation, deterministic build-time artifact/version injection); add 'none' to the releaseTrigger enum for repos with no release automation (WIP or file-history-only). Non-conformant branch states (missing main/develop, 'master', 'first-branch', stale/empty main) are recorded and flagged for correction, never treated as permanent. TelemetryImporter noted abandoned; .github noted as a candidate for removal.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CopilotAI review requested due to automatic review settings July 6, 2026 13:48
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

This PR updates the fleet classification spec and registry to clear the remaining backlog by (a) introducing an EDA-focused project type and (b) allowing repos to explicitly declare that they have no release automation, while moving 14 repos from backlog to cataloged with recorded drift notes.

Changes:

  • Add a new eda project type to the spec with detection heuristics and audit checks.
  • Extend the registry schema to allow releaseTrigger: none.
  • Reclassify 14 previously-backlog repositories as cataloged with types, release triggers, and drift notes.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

FileDescription
spec/project-types.jsonAdds the new eda type definition, including detection patterns and audit check assertions.
registry/repos.schema.jsonExtends releaseTrigger enum to include none.
registry/repos.jsonConverts 14 repos from backlog to cataloged and records their classification fields and drift notes.

Comment threadregistry/repos.json Outdated
@ptr727
ptr727 requested a review from CopilotJuly 6, 2026 13:51
@ptr727@claude
Correct NxWitness releaseTrigger to two-phase
f3d9abf
dispatch-only understated it: the repo publishes on the weekly schedule and a Matrix.json pin push and merges do not publish - the two-phase model plus a codegen-pin republish. Align the value with the driftNote. Addresses Copilot review.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment threadregistry/repos.json Outdated
CopilotAI review requested due to automatic review settings July 6, 2026 13:53
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

registry/repos.schema.json:29

  • The schema currently allows invalid publish combinations (e.g., target "github-release" with mechanism "oidc"/"static-secret", or non-github-release targets with mechanism "none"). Because spec/validate.py doesn't enforce a target-vs-mechanism match when targetMechanisms[target] is null (github-release), these mistakes can slip in silently; consider enforcing the allowed mechanism per target directly in the JSON schema.
 "releaseTrigger": { "enum": ["two-phase", "publish-on-merge", "dispatch-only", "none"] },
"mechanism": { "enum": ["oidc", "static-secret", "none"] },
"target": { "enum": ["nuget", "pypi", "docker", "github-release"] },
"repo": {
"type": "object",

Comment threadregistry/repos.json Outdated
@ptr727@claude
Drop implicit baseline App secrets from NxWitness requiredSecrets
a371a69
Per spec/secrets.json, baseline merge-bot App secrets (CODEGEN_APP_CLIENT_ID/PRIVATE_KEY) are implicit and not repeated in a repo's requiredSecrets, which lists only domain additions - matching VSCode-Server-DotNetCore and ESPHome-NonRoot (docker: DOCKER_HUB_* only). Addresses Copilot review.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ptr727
ptr727 requested a review from CopilotJuly 6, 2026 14:08
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

Comment threadspec/project-types.json Outdated
Comment threadregistry/repos.json Outdated
Comment threadregistry/repos.json Outdated
Comment threadregistry/repos.json Outdated
Comment threadregistry/repos.json Outdated
@ptr727@claude
Point groundTruthBranch at the populated branch; fix MediaTools trigg…
23a46d0
…er and wording
Per review: set groundTruthBranch to develop for KiCadLibrary/EspDinIoT/HolidayLights (main is stale/empty; the auditor must read the branch that holds the authoritative content, with driftNotes flagging that main should be promoted). Set MediaTools releaseTrigger to none (its two-phase publisher never fires - main/develop do not exist). Drop the hyphen in 'GitHub release' for consistency.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ptr727
ptr727 requested a review from CopilotJuly 6, 2026 14:14
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment threadspec/project-types.json Outdated
@ptr727@claude
Fix the D3 WORKFLOW.md anchor to the triple-hyphen heading slug
48764b7
The eda.build.deterministic ref (and a pre-existing occurrence) used #d3-versioning-and-classification; the heading 'D3 - Versioning and Classification' slugs to #d3---versioning-and-classification. Addresses Copilot review.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ptr727
ptr727 requested a review from CopilotJuly 6, 2026 14:18
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment threadspec/project-types.json
@ptr727@claude
Correct the D4 and D7 WORKFLOW.md anchors to their heading slugs
ae6c31c
'D4 - Release / Publish' and 'D7 - Concurrency, Permissions, Safety' slug to #d4---release--publish and #d7---concurrency-permissions-safety (triple hyphen for ' - ', double for ' / '). Fixes the broken single-hyphen forms (including my eda.release.zip ref) across the file. Addresses Copilot review.
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ptr727
ptr727 requested a review from CopilotJuly 6, 2026 14:25
CopilotAI reviewed Jul 6, 2026
View reviewed changes

CopilotAI 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.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

@ptr727
ptr727 merged commit 03e61d3 into developJul 6, 2026
7 checks passed
@ptr727
ptr727 deleted the feature/catalog-backlog-repos branch July 6, 2026 15:06
@ptr727ptr727 mentioned this pull request Jul 7, 2026
Merged
ptr727 added a commit that referenced this pull request Jul 7, 2026
@ptr727
Promote develop to main (#252)
8b9bd14
Two-phase promotion of the accumulated governance work on `develop` to
`main`. Carries #244-#251:
- #244 Foundational Principles (the model's governing *why*)
- #245 Replace the mypy ban with a strong-typing policy
- #246 aiopurpleair + homeassistant-purpleair audit reports
- #247 Catalog the 14 backlog repos; add the `eda` type and
`releaseTrigger: none`
- #248 Branch-conditional Docker platforms (multi-arch main, amd64
elsewhere)
- #249 Retire the both-branch matrix for symmetric single-branch
self-release
- #250 Declare workflow YAML as LF and validate line endings in CI
- #251 Extend line-ending governance to pattern files and catalog
snippets
All commits were reviewed and CI-green on their individual develop PRs.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ptr727