Skip to content

refactor(store): reduce Sonar complexity - #26

Merged
aksOps merged 14 commits into
mainfrom
fix/sonar-g005-store
Aug 1, 2026
Merged

refactor(store): reduce Sonar complexity#26
aksOps merged 14 commits into
mainfrom
fix/sonar-g005-store

Conversation

@aksOps

@aksOps aksOps commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • decompose store search, settings, secret publication, forge source, markdown import, and board transaction paths into package-local helpers
  • preserve SQL scope/order, credential-origin clearing, secret durability, receipt/CAS precedence, canonical identity, timestamps, labels, and tombstones
  • add regression coverage for publication order, rollback, equal-title identity, and strict empty-patch behavior
  • leave patchTask production code unchanged for a separate evidence-backed ACCEPTED disposition

Expected Sonar delta

Closes nine Go production complexity findings:

  • AZ-8DLJxETN2Y3sJmEuI
  • AZ-8DLJZETN2Y3sJmEt8
  • AZ-8DLJZETN2Y3sJmEt9
  • AZ-8DLKWETN2Y3sJmEuR
  • AZ-8DLKOETN2Y3sJmEuQ
  • AZ-8DLJnETN2Y3sJmEuB
  • AZ-8DLJnETN2Y3sJmEuC
  • AZ-8DLJnETN2Y3sJmEuD
  • AZ-8DLJnETN2Y3sJmEuE

Verification

  • 905/905 frontend tests; coverage lines 99.43%, branches 95.21%
  • frontend build passed
  • all Go packages passed
  • all Go packages passed uncached under the race detector
  • Go vet, staticcheck, repository formatting, and diff checks passed
  • all 14 commits have valid ED25519 signatures
  • independent integrated review approved with no findings
  • final PR Sonar scan: quality gate passed, 0 new issues, 97.2% new-code coverage, 0.0% duplication

@sonarqubecloud

sonarqubecloud Bot commented Aug 1, 2026

Copy link
Copy Markdown

@aksOps
aksOps merged commit 5783d56 into main Aug 1, 2026
9 checks passed
@aksOps
aksOps deleted the fix/sonar-g005-store branch August 1, 2026 21:10
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