[madalin-antonoiu-sonarsource]

Summary

Keep Architecture commands available for SonarQube Cloud and reject SonarQube Server connections with a clear login hint. The help and Architecture skill now describe this limitation.

This removes the Server version check and upgrade hint until the released Server APIs have been verified.

Testing

• Architecture integration tests, including Cloud access and Server rejection for every command
• Architecture unit tests, typecheck, lint and formatting

[netlify]

✅ Deploy Preview for sonarqube-cli canceled.

Name	Link
🔨 Latest commit	18d0e3e
🔍 Latest deploy log	https://app.netlify.com/projects/sonarqube-cli/deploys/6af33c60704d077c0efd531c

[hashicorp-vault-sonar-prod]

CLI-1523

[gitar-bot]

Code Review ✅ Approved 4 closed / 4 findings

🟡 Medium risk · Restricting architecture commands to SonarQube Cloud may prevent Server users from reading or updating architecture data through the CLI.

Keeps Architecture commands available for SonarQube Cloud and rejects SonarQube Server connections with a clear login hint, with help text and the Architecture skill updated to describe the Cloud-only limitation. Stale Server-path test assertions and the root help snapshot were updated to match, and no open issues remain.

✅ 4 closed

✅ Quality: Unreachable Server branches still assert removed Server guidance

📄 tests/integration/specs/architecture/architecture-findings.test.ts:413-425 📄 tests/integration/specs/architecture/architecture-compatibility.test.ts:274-283
The new if (!cloud) { …; return; } guards make the platform conditionals that follow them dead. In architecture-findings.test.ts, else expect(result.stderr).toContain('older Server versions') can never run. It also asserts text this PR deletes from collectionFailure. In architecture-compatibility.test.ts, if (cloud) expect(...).not.toContain('Server build') is now always true. These leftovers describe a contract that no longer exists, so they would fail or mislead if Server support came back. Remove the dead conditionals and keep only the Cloud assertions after the early return.

✅ Cross-PR: #1217's Server-path tests fail once Cloud-only gating lands

📄 src/commands/architecture/register.ts:62-71 📄 tests/integration/specs/architecture/architecture-compatibility.test.ts:274-283
Breaks when this PR and #1217 both merge into master and #1217's Server-path architecture integration tests run against the combined code. Merge order does not matter.

How

1. requireCloud (src/commands/architecture/register.ts:62-71) runs first in every architecture action and rejects any connection that is not Cloud with 'requires a SonarQube Cloud connection'.
2. SONARCH-3492 CLI: Report and respect a file-managed intended architecture #1217 adds compatibility tests that loop over cloud of [false, true] and expect Server-specific output. For example, tests/integration/specs/architecture/architecture-compatibility.test.ts expects toContain('Server build') when !cloud.
3. SONARCH-3492 CLI: Report and respect a file-managed intended architecture #1217 also adds Server hints to errors.ts (collectionFailure's 'older Server versions' text) and to sonar-architecture-api.ts. This PR removes or bypasses those hints, so the errors.ts hunks conflict and the Server branches can no longer be reached from the CLI.

Fix

• This PR: nothing needed; the Cloud-only gate is intended.
• SONARCH-3492 CLI: Report and respect a file-managed intended architecture #1217: rebase onto the Cloud-only gating. Change the !cloud cases to expect the Cloud-connection rejection, as this PR does, and drop the re-added Server hints in errors.ts/sonar-architecture-api.ts.

Compared 05caa7f with #1217 at ba16777

✅ Bug: Root help snapshot test still expects old architecture description

📄 src/commands/architecture/register.ts:77
This commit changes the architecture description to 'Read and set the architecture of a project (SonarQube Cloud only)'. Root help builds each summary from command.description() (src/core/commands/root-help.ts), so the root help output changes. tests/integration/specs/help/root-help.test.ts:60 still expects the old line, and lines 97/108/109 compare the whole output with toBe(getExpectedRootHelp()), so the sonar, sonar --help and bare root-help tests will fail. To fix it, update the expected line to the new description.

Closed: Quality: Generated docs still show the architecture description without Cloud-only

📄 src/commands/architecture/register.ts:77
docs/data/commands.json and docs/llms.txt are generated from command descriptions by build-scripts/docs/generate-docs.ts (bun run gen:docs). Both still show 'Read and set the architecture of a project' without the new '(SonarQube Cloud only)' suffix. Other Cloud-only commands (organizations, remediate, import) do show it there. The published machine-readable docs therefore don't mention the restriction this PR adds. To fix it, run bun run gen:docs and commit the regenerated files.

Verification Rules 2/2 met

Functional validation No results

Rules 2 passed 2 not applicable

✅ Internal artifact registries
✅ Repository license compliance

Auto-approval Withheld

_{For more verification details, comment gitar display:verbose.}

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply	Compact
gitar auto-apply:on	gitar display:verbose

_{Was this helpful? React with 👍 / 👎 | Gitar}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues
0 New dependency risks

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud