Skip to content

feat(security): enable code scanning, which has never run here - #4

Merged
rldyourmnd merged 1 commit into
mainfrom
feat/enable-code-scanning
Aug 23, 2026
Merged

feat(security): enable code scanning, which has never run here#4
rldyourmnd merged 1 commit into
mainfrom
feat/enable-code-scanning

Conversation

@rldyourmnd

Copy link
Copy Markdown
Contributor

GET /code-scanning/analyses returns no analysis found for this repository and default setup is not-configured. Go is 100% of this repository.

It is not one repository

Repository Code Scanned
github-device-sync Go 97% nothingNDDev-OpenNetwork/github-device-sync#25
github-actions Go 95% nothing
ci-workflows Python 98% actions, python
cd-workflows Python 75% nothing
macos-ubuntu-bootstrap Python 62% actions only — NDDev-OpenNetwork/macos-ubuntu-bootstrap#7
agent-runtime Go 100% nothing

Four of six have never been scanned, and they are the entire Go control plane. The republication carried the code to the new organisation and left the scanning behind. Nothing failed, because nothing was watching — the same shape as NDDev-OpenNetwork/github-device-sync#22, #23 and macos-ubuntu-bootstrap#7.

Secret scanning and push protection were disabled on all six as well; both are free for public repositories and are now on.

The change

Advanced setup, not default setup: the caller is pinned, reviewable in a diff, and names its runner. The reusable's runner default belongs to the pinned commit rather than to this repository, so inheriting it would let a pin bump move fork pull requests onto private infrastructure with no diff here to review.

Pinned to 0.1.4 at the commit that tag actually resolves to. Worth stating: the estate's other twelve call sites pin an untagged 2026-08-15 commit while claiming version 0.13.3, which has never existed — NDDev-OpenNetwork/ci-workflows#45.

Deliberately left out of required_contexts. A check with no green run yet should not gate the pull request introducing it. Promote it once it has passed on main.

Verification

This repository's own commands, from .gds/repository.yaml:

  • go vet ./... → clean
  • go test -race ./... → every package ok
  • go build ./cmd/agent-runtime → ok
  • actionlint 1.7.12 → clean

Expect first-run findings. That is the point.

`GET /code-scanning/analyses` returns `no analysis found` for this repository
and default setup is `not-configured`. Go is 100%.

Four of the six public modules are in this state — github-device-sync,
github-actions, cd-workflows and agent-runtime — and they are the whole Go
control plane. The republication carried the code to the new organisation and
left the scanning behind; nothing failed, because nothing was watching.

Advanced setup rather than default setup, so the caller is pinned, reviewable in
a diff, and names its runner explicitly: the reusable's `runner` default belongs
to the pinned commit, not to this repository, so inheriting it would let a pin
bump move fork pull requests onto private infrastructure with no diff here.

Pinned to `0.1.4` at the commit that tag actually resolves to. The estate's
other twelve call sites claim a version `0.13.3` that has never existed, on an
untagged 2026-08-15 commit — NDDev-OpenNetwork/ci-workflows#45.

Left out of `required_contexts` deliberately: a check with no green run yet
should not gate the pull request that introduces it. Promote it once it has
passed on `main`.

Verified with this repository's own commands: `go vet ./...` clean,
`go test -race ./...` every package ok, the declared build ok, actionlint 1.7.12
clean.
@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.

@rldyourmnd
rldyourmnd merged commit 0a831ad into main Aug 23, 2026
7 checks passed
@rldyourmnd
rldyourmnd deleted the feat/enable-code-scanning branch August 23, 2026 22:15
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