Skip to content

ci: add code scanning and require code owners - #9

Merged
kriptoburak merged 3 commits into
mainfrom
codex/add-codeowners
Jul 28, 2026
Merged

ci: add code scanning and require code owners#9
kriptoburak merged 3 commits into
mainfrom
codex/add-codeowners

Conversation

@kriptoburak

@kriptoburak kriptoburak commented Jul 28, 2026

Copy link
Copy Markdown
Member

Summary

  • require code-owner review for every path
  • add pinned CodeQL analysis for repository scripts
  • add pinned OpenSSF Scorecard reporting

Security Design

  • actions use immutable commit pins
  • checkouts never persist credentials
  • jobs use minimal scoped permissions
  • Scorecard runs only after trusted pushes and schedules
  • CodeQL uses extended security queries

Validation

  • both listed owners have write permission
  • protected-branch review settings remain enforced
  • actionlint passes
  • Node syntax checks pass
  • git diff --check passes

Note

Add CodeQL scanning, OpenSSF Scorecard, and CODEOWNERS to the repository

  • Adds a CodeQL workflow that runs JavaScript/TypeScript security analysis on push and pull requests to main and on a weekly schedule, reporting results to GitHub code scanning.
  • Adds an OpenSSF Scorecard workflow that runs on push to main and weekly, publishing SARIF results to GitHub code scanning and retaining them as artifacts for 5 days.
  • Adds a CODEOWNERS file assigning repository-wide review requirements to @kriptoburak and @furkanerday.

Macroscope summarized 23b67d5.

@kriptoburak
kriptoburak requested a review from furkanerday July 28, 2026 16:51
@kriptoburak
kriptoburak enabled auto-merge (squash) July 28, 2026 16:53
@kriptoburak kriptoburak changed the title ci: require code owner review ci: add code scanning and require code owners Jul 28, 2026
@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@furkanerday furkanerday left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the complete diff, description, commit history, checks, discussion, repository policy, relevant contracts, tests, dependency and release integrity, licensing, and public-safety implications. Required checks are green and no actionable review threads remain.

@kriptoburak
kriptoburak merged commit 0277b89 into main Jul 28, 2026
7 checks passed

@furkanerday furkanerday left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Human-authored commits 1566abd, d02ad62, 23b67d5 lack author-matching Signed-off-by trailers. Each author must amend their own affected commit and re-push it; a later maintainer commit cannot cure an earlier contributor commit. Re-request review after the rewritten commit history and required checks complete.

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.

3 participants