Skip to content

Verify catalog source status with author details - #24

Merged
flemming-n-larsen merged 3 commits into
mainfrom
fix/catalog-status-creator
Oct 21, 2026
Merged

flemming-n-larsen merged 3 commits into
mainfrom
fix/catalog-status-creator

Conversation

@flemming-n-larsen

@flemming-n-larsen flemming-n-larsen commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

The first live catalog PR was opened successfully by the Publisher App, but verification escalated because GitHub's combined commit-status response omits the status creator. The verifier now reads the chronological status list, checks the latest bot-moderation status and its Moderator App author, and rejects missing or untrusted authors.

The implementation-validation workflow also claimed every unpublished bot update for its PR author. A generic validation mode now runs all bot checks without ownership attribution; submission moderation retains the ownership check, and catalog generation cannot use the generic mode. This lets implementation fixes pass while catalog publication is pending.

GOVERNANCE.md records the Publisher App ID, permissions, secrets, and live evidence. After merge, rerun catalog publication for merged test submission #22. Failed catalog PR #23 was closed because it has an old base; the replacement should verify and merge automatically.

Validation: 47 Python tests passed (2 platform skips); git diff --check passed. Live evidence: bot PR #22 auto-merged and catalog PR #23 was opened by the Publisher App.

@tank-royale-rumble-moderator

Copy link
Copy Markdown
Contributor

@flemming-n-larsen Moderation needs your decision for 82d34a1b8969: Non-bot or protected file changed: GOVERNANCE.md

The required check stays pending. After reviewing, use /moderation approve <reason> or request changes.

@tank-royale-rumble-moderator

Copy link
Copy Markdown
Contributor

@flemming-n-larsen Moderation needs your decision for 2e071439a543: Non-bot or protected file changed: .github/workflows/validate.yml

The required check stays pending. After reviewing, use /moderation approve <reason> or request changes.

@flemming-n-larsen
flemming-n-larsen merged commit 9f5c8b7 into main Oct 21, 2026
3 of 4 checks passed
@flemming-n-larsen
flemming-n-larsen deleted the fix/catalog-status-creator branch October 21, 2026 18:52
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