chore: add tui-design-system agent skill #3
elkaixAnnotations
15 errors and 14 warnings
|
lint
Process completed with exit code 1.
|
|
typescript-eslint(no-misused-spread):
apps/pythinker-code/src/tui/components/chrome/pythinker-logo.ts#L114
Using the spread operator on a string can mishandle special characters, because it produces Unicode code points, which will break complex characters (like emojis) into multiple parts.
|
|
eslint(no-unused-vars):
apps/pythinker-code/src/tui/components/chrome/welcome-banner.ts#L229
Parameter 'index' is declared but never used. Unused parameters should start with a '_'.
|
|
typecheck:
apps/pythinker-code/test/tui/components/chrome/welcome-banner.test.ts#L49
Property 'TERM' comes from an index signature, so it must be accessed with ['TERM'].
|
|
typecheck:
apps/pythinker-code/test/tui/components/chrome/welcome-banner.test.ts#L44
Property 'TERM' comes from an index signature, so it must be accessed with ['TERM'].
|
|
typecheck:
apps/pythinker-code/test/tui/components/chrome/welcome-banner.test.ts#L40
Property 'TERM' comes from an index signature, so it must be accessed with ['TERM'].
|
|
typecheck:
apps/pythinker-code/test/tui/components/chrome/pythinker-logo.test.ts#L70
Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
|
|
typecheck:
apps/pythinker-code/test/tui/components/chrome/pythinker-logo.test.ts#L41
Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
|
|
typecheck:
apps/pythinker-code/test/tui/components/chrome/pythinker-logo.test.ts#L41
Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
|
|
typecheck:
apps/pythinker-code/src/tui/components/chrome/welcome-banner.ts#L77
Property 'TERM' comes from an index signature, so it must be accessed with ['TERM'].
|
|
typecheck:
apps/pythinker-code/src/tui/components/chrome/pythinker-logo.ts#L245
Argument of type 'string | undefined' is not assignable to parameter of type 'string'.
|
|
typecheck:
apps/pythinker-code/src/tui/components/chrome/pythinker-logo.ts#L223
Object is possibly 'undefined'.
|
|
typecheck:
apps/pythinker-code/src/tui/components/chrome/pythinker-logo.ts#L217
Object is possibly 'undefined'.
|
|
test
Process completed with exit code 1.
|
|
[acp-adapter] test/convert.test.ts > acpBlocksToPromptParts > URL-decodes file:// paths (spaces, unicode):
packages/acp-adapter/test/convert.test.ts#L153
AssertionError: expected [ { type: 'text', …(1) }, …(1) ] to deeply equal [ { type: 'text', …(1) }, …(1) ]
- Expected
+ Received
@@ -2,9 +2,9 @@
{
"text": "/Users/a b/foo.ts",
"type": "text",
},
{
- "text": "/Users/i18n/foo.ts",
+ "text": "/Users/中文/foo.ts",
"type": "text",
},
]
❯ test/convert.test.ts:153:17
|
|
lint
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
eslint-plugin-unicorn(catch-error-name):
packages/agent-core/src/di/util/idleValue.ts#L106
The catch parameter "err" should be named "error"
|
|
eslint(no-else-return):
packages/agent-core/src/di/util/idleValue.ts#L47
Unnecessary `else` after `return`.
|
|
eslint-plugin-unicorn(prefer-string-replace-all):
packages/protocol/src/asyncapi.ts#L135
Prefer `String#replaceAll()` over `String#replace()` when using a regex with the global flag.
|
|
eslint-plugin-unicorn(prefer-string-replace-all):
packages/protocol/src/asyncapi.ts#L135
Prefer `String#replaceAll()` over `String#replace()` when using a regex with the global flag.
|
|
eslint-plugin-unicorn(catch-error-name):
packages/agent-core/src/errors/unexpectedError.ts#L70
The catch parameter "err" should be named "error"
|
|
eslint-plugin-unicorn(catch-error-name):
packages/agent-core/src/errors/unexpectedError.ts#L56
The catch parameter "handlerErr" should be named "error"
|
|
eslint-plugin-unicorn(prefer-math-trunc):
packages/agent-core/src/tools/cron/jitter.ts#L74
Prefer `Math.trunc()` over instead of `>>> 0`.
|
|
eslint-plugin-unicorn(prefer-math-trunc):
packages/agent-core/src/tools/cron/jitter.ts#L71
Prefer `Math.trunc()` over instead of `| 0`.
|
|
eslint-plugin-unicorn(prefer-code-point):
packages/agent-core/src/tools/cron/jitter.ts#L71
Prefer `codePointAt` over `charCodeAt`
|
|
eslint-plugin-unicorn(prefer-string-replace-all):
packages/kaos/src/internal.ts#L208
Prefer `String#replaceAll()` over `String#replace()` when using a regex with the global flag.
|
|
typecheck
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
build
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
test
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|