Add JS client Node.js troubleshooting skill#25
Conversation
| @@ -0,0 +1,52 @@ | |||
| --- | |||
| name: clickhouse-js-node-troubleshooting | |||
There was a problem hiding this comment.
I love this - but I'd like to know why this cannot be a more general skill? This could then apply to more language clients such as java, rust, go, etc? We could then write a judge harness for all clients. Curious your thoughts on this too @mshustov
I think I envision something like this. where all have similar rules, and more specific ones like you have here.
skills
clickhouse-client-js
- rule1
- rule2
clickhouse-client-rust
- rule1
- rule2
clickhouse-client-go
- rule1
- rule2
...etc
There was a problem hiding this comment.
This is something to think about, thanks! The reason in short is simple: task focussed skills allow for more tokens in the index file.
There was a problem hiding this comment.
@pjhampton How would you suggest us to layout the skills?
skills
usage
troubleshooting
clickhouse-client-js
- usage
- rule-1
- rule-2
- troubleshooting
- rule-1
- rule-2
- ...
There was a problem hiding this comment.
Pull request overview
Adds a new “ClickHouse Node.js client troubleshooting” skill, providing ready-to-use guidance for diagnosing common @clickhouse/client issues (TLS, keep-alive/ECONNRESET, query params, compression, logging, etc.).
Changes:
- Introduces
clickhouse-js-node-troubleshootingskill entrypoint with an issue index. - Adds multiple reference pages covering common Node.js client failure modes and configuration pitfalls.
- Provides copy/paste examples and version-gated tips for
@clickhouse/client.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| skills/clickhouse-js-node-troubleshooting/SKILL.md | Skill definition and issue index routing users to reference docs. |
| skills/clickhouse-js-node-troubleshooting/reference/tls.md | TLS/mTLS setup and common Node certificate errors. |
| skills/clickhouse-js-node-troubleshooting/reference/socket-hangup.md | ECONNRESET triage focused on dangling streams and keep-alive tuning. |
| skills/clickhouse-js-node-troubleshooting/reference/readonly-users.md | Explains readonly user limitation with response compression + fix. |
| skills/clickhouse-js-node-troubleshooting/reference/query-params.md | Parameter binding syntax guidance + SQL-injection warning. |
| skills/clickhouse-js-node-troubleshooting/reference/proxy-pathname.md | Correct pathname usage for reverse proxies / path prefixes. |
| skills/clickhouse-js-node-troubleshooting/reference/logging.md | Enabling logging and implementing a custom logger. |
| skills/clickhouse-js-node-troubleshooting/reference/data-types.md | Common JSON format type pitfalls + format selection table. |
| skills/clickhouse-js-node-troubleshooting/reference/compression.md | Request/response compression behavior, defaults, and pitfalls. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Agent-Logs-Url: https://github.com/ClickHouse/agent-skills/sessions/7d87ea2c-28c4-40c7-94e2-b843d60b51f7 Co-authored-by: peter-leonov-ch <209667683+peter-leonov-ch@users.noreply.github.com>
|
|
Source: