Skip to content

Fix reference-to-video review findings #14318

Fix reference-to-video review findings

Fix reference-to-video review findings #14318

Triggered via pull request September 10, 2026 10:14
Status Success
Total duration 15m 43s
Artifacts 3

test.yml

on: pull_request
Quality Gate (npm run check)  /  build
4m 7s
Quality Gate (npm run check) / build
Quality Gate (npm run check)  /  changes
25s
Quality Gate (npm run check) / changes
Matrix: Quality Gate (npm run check) / static
Workflow Integration Tests
4m 51s
Workflow Integration Tests
Workflow Runner Browser E2E
5m 56s
Workflow Runner Browser E2E
Matrix: Quality Gate (npm run check) / built
Quality Gate (npm run check)  /  docker
7m 55s
Quality Gate (npm run check) / docker
Quality Gate (npm run check)  /  quality
4s
Quality Gate (npm run check) / quality
Fit to window
Zoom out
Zoom in

Annotations

10 warnings and 1 notice
keyboard(no-window-key-listener): web/src/components/sketch/useEditorKeyboardShortcuts.ts#L220
web/src/components/sketch/useEditorKeyboardShortcuts.ts:220:5: Register a combo in KeyPressedStore instead of listening on window; see web/src/stores/AGENTS.md § Keyboard
eslint(curly): mobile/src/stores/ChatStore.ts#L242
mobile/src/stores/ChatStore.ts:242:18: Expected { after 'if' condition.
eslint(no-useless-assignment): packages/storage/src/file-storage-adapter.ts#L43
packages/storage/src/file-storage-adapter.ts:43:7: This assigned value is not used in subsequent statements.
eslint(no-control-regex): packages/node-sdk/src/sandbox-pack-discovery.ts#L51
packages/node-sdk/src/sandbox-pack-discovery.ts:51:30: Unexpected control characters
eslint(no-control-regex): packages/protocol/src/sandbox-package.ts#L77
packages/protocol/src/sandbox-package.ts:77:16: Unexpected control characters
eslint(no-control-regex): packages/protocol/src/sandbox-package.ts#L11
packages/protocol/src/sandbox-package.ts:11:30: Unexpected control characters
react-hooks(exhaustive-deps): packages/cli/src/app.tsx#L966
packages/cli/src/app.tsx:966:6: React Hook useCallback has unnecessary dependency: enabledTools
react-hooks(exhaustive-deps): packages/cli/src/app.tsx#L832
packages/cli/src/app.tsx:832:58: React Hook useCallback has missing dependencies: 'threadId', 'buildTools', 'registry', and 'agentProviders'
react-hooks(exhaustive-deps): packages/cli/src/app.tsx#L454
packages/cli/src/app.tsx:454:7: React Hook useEffect has a missing dependency: 'addMessage'
react-hooks(exhaustive-deps): packages/cli/src/app.tsx#L407
packages/cli/src/app.tsx:407:15: React Hook useEffect has a missing dependency: 'addMessage'
🎭 Playwright Run Summary
27 passed (23.5s)

Artifacts

Produced during runtime
Name Size Digest
nodetool-ai~nodetool~WZ7L0H.dockerbuild
107 KB
sha256:b41d0621dcc19fda7de43da1575018bc572e017fd10c3e4230120a1fdeff82f3
packages-dist Expired
12.8 MB
sha256:da9c4e7c73d712cf279cc50725c6a3b10804321652edcd04be46d528daa4e129
reliability-harness-dist Expired
168 KB
sha256:0489c26b32a867758753d61d926ff20550138e1cab211c88f510f6153efe28a6