Skip to content

fix(sonar): point at this repo's own SonarCloud project#20

Merged
hyperpolymath merged 2 commits into
mainfrom
fix/sonar-project-key
Jul 21, 2026
Merged

fix(sonar): point at this repo's own SonarCloud project#20
hyperpolymath merged 2 commits into
mainfrom
fix/sonar-project-key

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

sonar-project.properties declares sonar.projectKey=hyperpolymath_rsr-template-repo — the key belonging to a different repository, copied verbatim when this repo was derived from rsr-template-repo. This points it at this repo's own SonarCloud project, which already exists.

Why this matters now (and why it's worth doing before the token is added)

The SonarQube workflow here currently fails on every run — but not because of the key. SONAR_TOKEN is not set on this repo, so the scanner aborts at authentication:

SONAR_TOKEN:
##[warning]Running this GitHub Action without SONAR_TOKEN is not recommended
ERROR Not authorized or project not found. Please check the 'SONAR_TOKEN' environment
      variable, the 'sonar.projectKey' and 'sonar.organization' properties, ...

Because auth fails first, nothing is currently being submitted anywhere — the wrong key is latent, not actively corrupting data. But the moment SONAR_TOKEN is added, this repo would start pushing its analysis into hyperpolymath_rsr-template-repo, commingling with every other repo carrying the same copied key. Fixing the key first avoids polluting that project.

Verified

  • The target project exists on SonarCloud (checked via the public API).
  • hyperpolymath_rsr-template-repo also exists — display name -REPO- — which is why the misdirected scans would land somewhere real rather than erroring out.
  • sonar.organization is unchanged and was already correct.

Found during an estate-wide sweep of sonar-project.properties files; several other repos carry the same copied key and need the same one-line fix.

Test plan

  • Only sonar.projectKey and the project URL in the header comment change
  • Target project confirmed to exist on SonarCloud
  • Maintainer to add the SONAR_TOKEN repository secret — the workflow cannot pass without it regardless of this change

🤖 Generated with Claude Code

sonar.projectKey was hyperpolymath_rsr-template-repo, copied verbatim from
the template repo this one was derived from. The correct project for this
repo already exists on SonarCloud.

The workflow currently fails at authentication (no SONAR_TOKEN secret), so
nothing is being mis-submitted today -- but once the token is added this
repo would start reporting into the template's project. Fix the key first.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
hyperpolymath added a commit that referenced this pull request Jul 21, 2026
…h (PR-6, closes the sequence) (#21)

Final PR of the completion sequence (#11#12#13#15#16#17 →
this).

- **STATE.a2ml** — completion 15→32 with per-phase justification
comments; blockers echo #18/#19 + the anytype audit trail (anytype
#13#19, PR #20).
- **AFFIRMATION.adoc** — anchor filled (main @ `a11c9877`,
2026-07-21T14:43Z, Lean 4.32.0/elan 4.2.3) in the same session that
re-ran every check listed; every "affirmed-ran" row corresponds to a
command actually executed. NOT-claims list keeps ET-2 closure, ET-6..9,
ET-10..13, ET-14 explicitly open.
- **Justfile** — `test`/`lint`/`fmt-check` were template echo-stubs
(fake green). Now: `test` = proof gate + drift gate; `lint` =
dangerous-construct scan, md-in-docs, root-shape, template validation;
`fmt-check` honestly reports that nothing is checked.
`check-no-vlang.sh` deferred to #19 (tracked, not silently skipped).
`.machine_readable/root-allow.txt` updated for root entries landed by
#14/#17.
- **README** — badge `theory_(no_proofs_yet)` → `theory_(L2_proven,
L1_core_proven, rest_OPEN)`; Status section states precisely what is
machine-checked vs open.

Verification (all run in this session at the anchor SHA):
`./scripts/check-proofs.sh lean4` PASS (audit 12/12);
`./scripts/scan-dangerous.sh` PASS; `./scripts/check-proof-status.sh`
PASS; `./scripts/validate-template.sh` PASS (4 warnings); `just test &&
just quality` PASS end-to-end.

🤖 Generated with [Claude Code](https://claude.com/claude-code)
@hyperpolymath
hyperpolymath marked this pull request as ready for review July 21, 2026 14:52
@hyperpolymath
hyperpolymath merged commit ddf5749 into main Jul 21, 2026
18 of 33 checks passed
@hyperpolymath
hyperpolymath deleted the fix/sonar-project-key branch July 21, 2026 14:52
@sonarqubecloud

Copy link
Copy Markdown

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