[Bug] SonarQube: assorted single-site reliability bugs (S7497, S7930, S3516, S7727) #243

New Issue

2026-06-19T18:44:20Z

spikerj commented

2026-06-19 18:44:20 +00:00

Roll-up of single-occurrence RELIABILITY bugs surfaced by SonarQube, in our own code.

Items

python:S7497 (MAJOR) — asyncio.CancelledError must be re-raised after cleanup. spikersoft-backend/SpikerSoft.EventHandlers.ImageDescription.Python/image_description_service.py:961
Web:S7930 (CRITICAL) — duplicate HTML id="course-format" (first at line 40). projects/spikersoft/src/app/_components/syllabus/syllabus.component.html:69
typescript:S3516 (BLOCKER) — function always returns the same value (dead return contract). projects/spikersoft/src/app/_components/learning/git-playground/git-engine/wasm-git-engine.ts:124
typescript:S7727 (MAJOR) — don't pass mapWorkspaceRowToSavedObject directly to .map() (extra args leak in). libraries/features/dev-tools-blockly/src/lib/workspace.service.ts:230

Each is independent; check off as fixed.

Rules: https://rules.sonarsource.com/python/RSPEC-7497/ · https://rules.sonarsource.com/html/RSPEC-7930/ · https://rules.sonarsource.com/typescript/RSPEC-3516/ · https://rules.sonarsource.com/typescript/RSPEC-7727/

Roll-up of single-occurrence RELIABILITY bugs surfaced by SonarQube, in our own code. ## Items - [ ] `python:S7497` (MAJOR) — `asyncio.CancelledError` must be re-raised after cleanup. `spikersoft-backend/SpikerSoft.EventHandlers.ImageDescription.Python/image_description_service.py:961` - [ ] `Web:S7930` (CRITICAL) — duplicate HTML `id="course-format"` (first at line 40). `projects/spikersoft/src/app/_components/syllabus/syllabus.component.html:69` - [ ] `typescript:S3516` (BLOCKER) — function always returns the same value (dead return contract). `projects/spikersoft/src/app/_components/learning/git-playground/git-engine/wasm-git-engine.ts:124` - [ ] `typescript:S7727` (MAJOR) — don't pass `mapWorkspaceRowToSavedObject` directly to `.map()` (extra args leak in). `libraries/features/dev-tools-blockly/src/lib/workspace.service.ts:230` Each is independent; check off as fixed. _Rules: https://rules.sonarsource.com/python/RSPEC-7497/ · https://rules.sonarsource.com/html/RSPEC-7930/ · https://rules.sonarsource.com/typescript/RSPEC-3516/ · https://rules.sonarsource.com/typescript/RSPEC-7727/_

spikerj added the bug sonarqube labels 2026-06-19 18:44:20 +00:00

spikerj commented

2026-06-20 02:03:22 +00:00

Resolved across spikersoft-backend PR #12 (S7497 — python CancelledError re-raise) and spikersoft-angular PR #73 (S7930 duplicate id, S3516 single-return in attachRemote, S7727 explicit .map((w,i)=>…)). All four items done; both PRs merged to master. Closing.

Resolved across spikersoft-backend PR #12 (`S7497` — python `CancelledError` re-raise) and spikersoft-angular PR #73 (`S7930` duplicate id, `S3516` single-return in `attachRemote`, `S7727` explicit `.map((w,i)=>…)`). All four items done; both PRs merged to `master`. Closing.

spikerj closed this issue

2026-06-20 02:03:38 +00:00

spikerj referenced this issue

2026-07-12 19:37:58 +00:00

[infra drift] image-description & trellis3d deploy to /mnt/infrastructure folders that don't exist #403

spikerj referenced this issue

2026-07-12 19:43:31 +00:00

[infra drift] image-description & trellis3d deploy to /mnt/infrastructure folders that don't exist #403

Sign in to join this conversation.