[devin-ai-integration]

What and why

Clears 14 of the 15 open SonarCloud findings on develop in Go packages under internal/ outside the runtime, at the root; one is deferred because two open PRs rewrite the same function. Behaviour is unchanged: each edit is a split into named helpers, a named constant, a table that replaces a switch (with a new test that it names every kind), or a renamed test variable.

Rule	File:line on develop	What was done
go:S3776 (32)	internal/check/passes/typecheck.go:389 checkTypeTarget	Fixed: target resolution (feature-reference unwrap, chain checks, alias) is typeTargetSymbol; the kind-message diagnostic is reportTargetKind.
go:S3776 (33)	internal/doc/queryexec/operations.go:604 evaluateProject	Fixed: a projection holds the operation's columns and what reading rows learnt; newProjection reads the arguments, cells reads one row, checkProperties reports unknown/missing properties. Argument and error order unchanged.
go:S3776 (34)	internal/ir/view/general.go:588 packageEdges	Fixed: split into aliasEdges and importEdges.
go:S3776 (33)	internal/semantic/semantics/model.go:982 FeatureTypeSetWithImplied	Fixed: the usage-base lookup is withUsageBases.
go:S3776 (35)	internal/workspace/project/discovery.go:286 used	Deferred: #1568 (fix/usage-version-ranges) and #1561 (fix/usage-elimination-named-forms) both rewrite the body of used (their hunks span lines 302–380). Refactoring it under them would conflict with both; it should be split once they land, against the function they leave behind.
go:S1479	internal/ir/view/view.go:289 EdgeKind.String (34 branches)	Fixed: one edgeKindNames table sized by a new edgeKindCount sentinel. The switch had no exhaustiveness check (Go has none for switch; make lint runs no exhaustive), so nothing relied on one; the table adds one — TestEdgeKindNamesEveryKind fails on a kind added without a name, and a name out of range still answers "edge".
go:S1192	internal/exec/smt/encode.go:257 "attribute " ×3	Fixed: attributeConstruct constant.
go:S1192	internal/ir/view/dot.go:545 "arrowhead=vee" ×3	Fixed: the two literal uses now use the existing dotArrowheadVee constant.
godre:S8209	internal/ir/docplan/markup/asciidoc.go:135	Fixed: flat, nodes []asciiDocBlock.
godre:S8193	internal/doc/queryexec/named_test.go:143	Fixed: the error's Actual name is now reported in the failure message.
go:S978	internal/frontend/grpc/query_operators_test.go:170 any	Fixed: any → listed.
go:S978	internal/workspace/libs/loader_copies_test.go:19 copy	Fixed: copy → replica.
go:S978	internal/workspace/libs/loader_copies_test.go:84 copy	Fixed: same.

No dispositions.

How it was verified

• go build ./..., go vet on every touched package, gofmt -l empty.
• go test -count=1 -timeout 60m ./internal/check/... ./internal/doc/... ./internal/exec/smt/... ./internal/frontend/grpc/... ./internal/ir/... ./internal/semantic/... ./internal/workspace/...: all 37 packages ok.
• gocognit -over 30 on the four refactored files: none of the four functions remains over 30.
• Sonar's analyze_code_snippet on each changed file: the develop copies report exactly the findings above; the branch copies report none of them (the remaining hits are findings already resolved on develop). godre:* rules are not reported by the snippet analyser; checked by eye.

Overlap with open PRs

• fix(project): read a usage's versionConstraint as a semver range #1568 and fix(project): link a usage by elimination only when its resource spells a location #1561 touch internal/workspace/project/discovery.go; this PR leaves that file untouched (see the deferred row), so there is no conflict.
• Dry-merged in a scratch worktree at origin/develop (12e4fc3): this head (2dc10e4), then refactor(symbols): drop the unexported duplicate of UsageSymbolKind #1613 (refactor/usage-symbol-kind-dedup, 50db66b, the only open PR touching internal/semantic) — merges clean; go build ./..., go vet and go test ./internal/semantic/... ./internal/check/passes ./internal/ir/view pass on the merged tree.
• No open perf/* PR touches internal/exec/runtime, internal/semantic or internal/frontend/lsp at this time (open PRs re-listed before leaving draft).
• All six Sonar PRs (refactor(runtime): clear the Sonar findings in internal/exec/runtime #1606, refactor(scripts): clear the Sonar findings in the Python and shell scripts #1608, refactor(internal): clear the Sonar findings outside the runtime #1610, refactor(scripts): clear the Sonar findings in the JavaScript scripts #1612, refactor(vscode): clear the Sonar findings in the diagram webview #1614, refactor(mdk): clear the Sonar findings in the Magic MDK plugin #1616) merge together on develop cleanly: go build ./..., go vet ./... and gofmt -l clean on the combined tree.

Checklist

• make test and make lint pass locally
• Tests added or updated for the change
• Documentation extended where it already covers the surface (see CONTRIBUTING.md)
• Changelog entry added as changes/unreleased/<slug>.<section>.md, not as an edit to CHANGELOG.md — not needed: refactor only
• baselines regenerated and make docs-counts run if a gate count moved (compliance rows need nothing: the census is counted at docs build) — no count moved
• No internal work-item labels (waves, slices, F4, K5) in the body, docs, or changelog

Requested by: @HuiJun

[devin-ai-integration]

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

• Disable automatic comment, CI, and merge conflict monitoring