molecule-ai/hermes-agent
49
0
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases Wiki Activity

fix(ci): keep required contexts emitted #24

Merged
hongming merged 2 commits from fix/required-ci-always-emits into main 2026-05-22 09:05:41 +00:00
Conversation 2 Commits 2 Files Changed 4
+135 -22
hongming commented 2026-05-22 08:54:48 +00:00
Owner
Copy Link
Copy Source

Summary\n- remove trigger path filters from branch-protected required workflows so required contexts always emit\n- make Tests/test and Tests/e2e internally skip expensive pytest when the diff is not Python/package/test relevant\n- keep Contributor Attribution and Supply Chain Audit contexts emitted, with internal no-op paths for irrelevant diffs\n- remove unavailable macos-latest from the required Nix matrix and gate GitHub-only sticky comment actions to github.com\n\n## Why\nWorkflow-only PRs such as #23 were forced through the full Hermes unit suite, which took ~22 min and failed on unrelated flakes. At the same time, other branch-protected contexts used trigger paths and could fail to emit at all. This keeps branch protection deterministic while avoiding unrelated heavy work.\n\n## Verification\n- python3 YAML parse over .github/workflows/*.yml\n- python3 invariant check: required PR workflows have no trigger paths/path-ignore; Nix matrix is ubuntu-latest only\n- git diff --check

## Summary\n- remove trigger path filters from branch-protected required workflows so required contexts always emit\n- make Tests/test and Tests/e2e internally skip expensive pytest when the diff is not Python/package/test relevant\n- keep Contributor Attribution and Supply Chain Audit contexts emitted, with internal no-op paths for irrelevant diffs\n- remove unavailable macos-latest from the required Nix matrix and gate GitHub-only sticky comment actions to github.com\n\n## Why\nWorkflow-only PRs such as #23 were forced through the full Hermes unit suite, which took ~22 min and failed on unrelated flakes. At the same time, other branch-protected contexts used trigger paths and could fail to emit at all. This keeps branch protection deterministic while avoiding unrelated heavy work.\n\n## Verification\n- python3 YAML parse over .github/workflows/*.yml\n- python3 invariant check: required PR workflows have no trigger paths/path-ignore; Nix matrix is ubuntu-latest only\n- git diff --check
hongming added 1 commit 2026-05-22 08:54:48 +00:00
fix(ci): keep required contexts emitted
Supply Chain Audit / Scan PR for critical supply chain risks (pull_request) Successful in 22s
Details
Contributor Attribution Check / check-attribution (pull_request) Failing after 24s
Details
Tests / e2e (pull_request) Successful in 37s
Details
Tests / test (pull_request) Successful in 38s
Details
Nix / nix (ubuntu-latest) (pull_request) Successful in 6m27s
Details
e8f60d9b5e
app-fe added 1 commit 2026-05-22 08:57:07 +00:00
fix(ci): skip attribution check for workflow-only PRs
Contributor Attribution Check / check-attribution (pull_request) Successful in 32s
Details
Tests / e2e (pull_request) Successful in 30s
Details
Supply Chain Audit / Scan PR for critical supply chain risks (pull_request) Successful in 32s
Details
Tests / test (pull_request) Successful in 41s
Details
Nix / nix (ubuntu-latest) (pull_request) Successful in 6m24s
Details
8664520f14
core-devops approved these changes 2026-05-22 09:05:11 +00:00
core-devops left a comment
Member
Copy Link
Copy Source

Approving as core-devops: required CI is green and this PR unblocks required contexts.

Approving as core-devops: required CI is green and this PR unblocks required contexts.
infra-sre approved these changes 2026-05-22 09:05:35 +00:00
infra-sre left a comment
Member
Copy Link
Copy Source

Approving as infra-sre: required CI is green; change is CI-only and preserves required contexts.

Approving as infra-sre: required CI is green; change is CI-only and preserves required contexts.
hongming merged commit 11935bbda3 into main 2026-05-22 09:05:41 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
core-devops
infra-sre
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agent-dev-a agent-dev-b agent-pm agent-researcher agent-reviewer agent-reviewer-1 agent-reviewer-cr2 app-fe (Molecule AI · app-fe) app-lead (Molecule AI · app-lead) app-qa (Molecule AI · app-qa) claude-ceo-assistant claude-ci-reader core-be (Molecule AI · core-be) core-devops (Molecule AI · core-devops) core-fe (Molecule AI · core-fe) core-lead (Molecule AI · core-lead) core-offsec (Molecule AI · core-offsec) core-qa (Molecule AI · core-qa) core-security (Molecule AI · core-security) core-uiux (Molecule AI · core-uiux) cp-be (Molecule AI · cp-be) cp-lead (Molecule AI · cp-lead) cp-qa (Molecule AI · cp-qa) cp-security (Molecule AI · cp-security) cui (Zhanlin Cui) dev-lead (Molecule AI · dev-lead) devops-engineer documentation-specialist (Molecule AI · documentation-specialist) fullstack-engineer (Molecule AI · fullstack-engineer) godwin hongming hongming-ceo-delegated hongming-codex-laptop hongming-kimi-laptop hongming-pc2 hongming-personal infra-lead (Molecule AI · infra-lead) infra-runtime-be (Molecule AI · infra-runtime-be) infra-sre (Molecule AI · infra-sre) integration-tester (Molecule AI · integration-tester) molecule-code-reviewer plugin-dev (Molecule AI · plugin-dev) pm release-manager (Molecule AI · release-manager) sdk-dev (Molecule AI · sdk-dev) sdk-lead (Molecule AI · sdk-lead) sop-tier-bot (SOP Tier-Check Bot) technical-writer (Molecule AI · technical-writer) triage-operator (Molecule AI · triage-operator)
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: molecule-ai/hermes-agent#24
Delete Branch "fix/required-ci-always-emits"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?