Skip to content

Remove duplicate OpenSSF Scorecard workflow (scorecard-enforcer.yml)#618

Merged
hyperpolymath merged 1 commit into
mainfrom
claude/awesome-edison-3711ps
Jun 21, 2026
Merged

Remove duplicate OpenSSF Scorecard workflow (scorecard-enforcer.yml)#618
hyperpolymath merged 1 commit into
mainfrom
claude/awesome-edison-3711ps

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

What

Removes the legacy .github/workflows/scorecard-enforcer.yml.

Why

This repo runs the OpenSSF Scorecard scan twice on every push:

  • scorecard.yml — delegates to the shared hyperpolymath/standards
    scorecard reusable workflow (the modern, canonical path), and
  • scorecard-enforcer.yml — a legacy self-contained job that runs its own
    full Scorecard scan + SARIF upload again.

The governance Check Workflow Staleness gate explicitly requires this:

Remove legacy scorecard-enforcer.yml …

Deleting the enforcer leaves a single Scorecard scan (via the reusable
workflow) and removes the redundant Actions run.

Note (separate follow-up)

The same staleness gate also asks to refresh the Hypatia + Scorecard reusable
pins to the current hyperpolymath/standards SHA. That needs read access
to hyperpolymath/standards (not in this session's scope), so it's left as a
follow-up — the staleness check may stay red on those pin items until then.
This PR addresses only the duplicate-scan removal.

🤖 Generated with Claude Code

https://claude.ai/code/session_0196movYBRwNw8vkLRaaFoTu


Generated by Claude Code

@hyperpolymath hyperpolymath marked this pull request as ready for review June 21, 2026 02:14
@hyperpolymath hyperpolymath enabled auto-merge (squash) June 21, 2026 02:14
@hyperpolymath hyperpolymath disabled auto-merge June 21, 2026 02:14
@hyperpolymath hyperpolymath enabled auto-merge (squash) June 21, 2026 02:15
@hyperpolymath hyperpolymath disabled auto-merge June 21, 2026 02:15
@hyperpolymath hyperpolymath enabled auto-merge (squash) June 21, 2026 02:16
scorecard-enforcer.yml runs a second, self-contained OpenSSF Scorecard
scan that duplicates the scan already performed by scorecard.yml, which
delegates to the shared hyperpolymath/standards scorecard reusable
workflow. Running both executes the (expensive) Scorecard scan twice on
every push.

The governance "Check Workflow Staleness" gate explicitly requires
removing this legacy file ("Remove legacy scorecard-enforcer.yml").
The single reusable-workflow scan in scorecard.yml is retained.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0196movYBRwNw8vkLRaaFoTu
@hyperpolymath hyperpolymath force-pushed the claude/awesome-edison-3711ps branch from f660f32 to 0370fd1 Compare June 21, 2026 02:17
@github-actions

Copy link
Copy Markdown

🔍 Hypatia Security Scan

Findings: 41 issues detected

Severity Count
🔴 Critical 2
🟠 High 23
🟡 Medium 16

⚠️ Action Required: Critical security issues found!

View findings
[
  {
    "reason": "Action denoland/setup-deno@v2 needs attention",
    "type": "unpinned_action",
    "file": "publish-jsr.yml",
    "action": "pin_sha",
    "rule_module": "workflow_audit",
    "severity": "medium"
  },
  {
    "reason": "Issue in instant-sync.yml",
    "type": "secret_action_without_presence_gate",
    "file": "instant-sync.yml",
    "action": "peter-evans/repository-dispatch",
    "rule_module": "workflow_audit",
    "severity": "high"
  },
  {
    "reason": "Shell execution -- validate input before passing to shell (1 occurrences, CWE-78)",
    "type": "js_exec_sync",
    "file": "/home/runner/work/affinescript/affinescript/packages/affinescript-cli/mod.js",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "high"
  },
  {
    "reason": "Shell execution -- validate input before passing to shell (2 occurrences, CWE-78)",
    "type": "js_exec_sync",
    "file": "/home/runner/work/affinescript/affinescript/packages/affine-vscode/mod.js",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "high"
  },
  {
    "reason": "Shell execution -- validate input before passing to shell (1 occurrences, CWE-78)",
    "type": "js_exec_sync",
    "file": "/home/runner/work/affinescript/affinescript/affinescript-vite/src/affine-plugin-improved.js",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "high"
  },
  {
    "reason": "expect() in hot path (32 occurrences, CWE-754)",
    "type": "expect_in_hot_path",
    "file": "/home/runner/work/affinescript/affinescript/affinescriptiser/src/codegen/wasm_gen.rs",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "medium"
  },
  {
    "reason": "expect() in hot path (29 occurrences, CWE-754)",
    "type": "expect_in_hot_path",
    "file": "/home/runner/work/affinescript/affinescript/affinescriptiser/src/codegen/affine_gen.rs",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "medium"
  },
  {
    "reason": "unsafe block -- requires SAFETY comment (2 occurrences, CWE-676)",
    "type": "unsafe_block",
    "file": "/home/runner/work/affinescript/affinescript/runtime/src/panic.rs",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "medium"
  },
  {
    "reason": "unsafe block -- requires SAFETY comment (1 occurrences, CWE-676)",
    "type": "unsafe_block",
    "file": "/home/runner/work/affinescript/affinescript/runtime/src/alloc.rs",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "medium"
  },
  {
    "reason": "unsafe block -- requires SAFETY comment (3 occurrences, CWE-676)",
    "type": "unsafe_block",
    "file": "/home/runner/work/affinescript/affinescript/runtime/src/ffi.rs",
    "action": "flag",
    "rule_module": "code_safety",
    "severity": "medium"
  }
]

Powered by Hypatia Neurosymbolic CI/CD Intelligence

@hyperpolymath hyperpolymath disabled auto-merge June 21, 2026 02:19
@hyperpolymath hyperpolymath merged commit 85e3f0d into main Jun 21, 2026
18 checks passed
@hyperpolymath hyperpolymath deleted the claude/awesome-edison-3711ps branch June 21, 2026 02:20
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.

2 participants