Frun1na opened a new pull request, #4727:
URL: https://github.com/apache/rocketmq-dashboard/pull/4727
### Which Issue(s) This PR Fixes
### Brief Description
Three places in the spec name values the implementation does not define:
- **§6.2** documented the consumer client protocol as `REMOTING` / `GRPC`.
`Protocol` is
`gRPC, Remoting`, and the enum appendix of this very document already says
`gRPC`, `Remoting` —
so §6.2 contradicted both the code and itself.
- **Appendix, client languages.** The row listed the UI display labels
`C++`, `C#`, `Node.js`.
`ClientLanguage` is `Java, Go, Python, Rust, Cpp, CSharp, NodeJS, PHP`,
which is what the API
returns (`ClientConnectionVO.language`) — the UI maps those to the labels
(`web/src/pages/cluster/clients.tsx`: `Cpp: { label: 'C++' }`, …). The
other appendix rows list
the persisted values, so this row now does too.
- **LLM providers, §14.1 and appendix.** Both listed `openai` / `azure` /
`ollama` / `qwen`.
`LlmConfigService.PROVIDER_MODELS` defines `openai`, `azure`, `anthropic`,
`deepseek`, `tongyi`,
`ollama` and `bedrock`. `qwen` is a model name under `tongyi`, not a
provider key, and the four
missing providers are accepted by the settings API this table documents.
### How Did You Test This Change?
Documentation only, so the Maven and Vitest suites were not re-run. The
values were read off the
implementation:
```
$ cat .../common/domain/enums/Protocol.java
public enum Protocol {
gRPC, Remoting
}
$ cat .../common/domain/enums/ClientLanguage.java
public enum ClientLanguage {
Java, Go, Python, Rust, Cpp, CSharp, NodeJS, PHP
}
$ grep -n "Cpp\|CSharp\|NodeJS" web/src/pages/cluster/clients.tsx
Cpp: { color: 'geekblue', label: 'C++' },
CSharp: { color: 'magenta', label: 'C#' },
NodeJS: { color: 'lime', label: 'Node.js' },
$ grep -n
'"openai"\|"azure"\|"anthropic"\|"deepseek"\|"tongyi"\|"ollama"\|"bedrock"\|"qwen"'
\
.../ops/ai/LlmConfigService.java
42: OPENAI = "openai"; 43: ANTHROPIC = "anthropic"; 44:
DEFAULT_PROVIDER = "tongyi";
54: "azure", ... 66: "deepseek", ... 69: "tongyi", ... 82: "ollama",
... 86: "bedrock",
(no "qwen" provider key anywhere)
```
### Checklist
- [x] One coherent change; unrelated modifications are not bundled in
- [x] Commit subject follows Conventional Commits (`docs:`)
- [ ] Tests added or updated for non-trivial changes, test methods named
`...Test`
- [ ] New UI text has both Chinese and English entries under `web/src/i18n/`
- [ ] Architecture constraints stay green (`mvn test` runs the ArchUnit
checks)
- [ ] New source files carry the ASF license header
- [x] Documentation touched where behaviour changed (README / `docs/` /
in-app help)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
