Skip to content

fix(review): align reusable runtime pins #3818

fix(review): align reusable runtime pins

fix(review): align reusable runtime pins #3818

Triggered via pull request August 21, 2026 05:23
Status Success
Total duration 14m 15s
Artifacts

ci.yml

on: pull_request
validate
2m 48s
validate
Task change ledger Windows smoke
3m 23s
Task change ledger Windows smoke
Matrix: lint
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

34 warnings
lint (features): src/features/codex-runtime-profile/core/domain/resolveCodexRuntimeProfile.ts#L43
"auto" | "adapter" | "api" | "cli-sdk" | "codex-native" | "opencode-cli" is overridden by string in this union type
lint (features): src/features/codex-account/main/infrastructure/detectCodexLocalAccountArtifacts.ts#L54
Make sure the regex used here, which is vulnerable to super-linear runtime due to backtracking, cannot lead to denial of service
lint (features): src/features/codex-account/main/composition/createCodexAccountFeature.ts#L718
'rateLimitsReadFailure' will use Object's default stringification format ('[object Object]') when stringified
lint (features): src/features/codex-account/main/composition/createCodexAccountFeature.ts#L623
'unknown' overrides all other types in this union type
lint (features): src/features/change-review-history/main/infrastructure/ReviewDraftHistoryStore.ts#L1229
Remove the declaration of the unused '_expectedGeneration' variable
lint (features): src/features/change-review-history/main/infrastructure/ReviewDraftHistoryStore.ts#L1228
Remove the declaration of the unused '_expectedRevision' variable
lint (features): src/features/change-review-history/main/infrastructure/ReviewDraftHistoryStore.ts#L1082
Prefer using an optional chain expression instead, as it's more concise and easier to read
lint (features): src/features/change-review-history/main/infrastructure/ReviewDraftHistoryStore.ts#L873
Provide a compare function that depends on "String.localeCompare", to reliably sort elements alphabetically
lint (features): src/features/agent-graph/renderer/ui/buildTransientHandoffMessage.ts#L36
Refactor this code to not use nested template literals
lint (features): src/features/agent-attachments/core/domain/types.ts#L97
"anthropic" | "codex" | "opencode" | "unknown" is overridden by string in this union type
lint (main): mcp-server/src/index.ts#L177
Found realpathSync from package "fs" with non literal argument at index 0
lint (main): mcp-server/src/index.ts#L98
Generic Object Injection Sink
lint (main): mcp-server/src/index.ts#L70
Variable Assigned to Object Injection Sink
lint (main): mcp-server/src/controller.ts#L24
Generic Object Injection Sink
lint (main): src/main/ipc/review.ts#L1472
Missing return type on function
lint (main): src/main/ipc/extensions.ts#L28
No rule allowing this dependency was found. File is of type 'main'. Dependency is of type 'preload'
lint (main): src/main/ipc/crossTeam.ts#L105
'req.text ?? ''' will use Object's default stringification format ('[object Object]') when stringified
lint (main): src/main/ipc/crossTeam.ts#L101
'req.toTeam ?? ''' will use Object's default stringification format ('[object Object]') when stringified
lint (main): src/main/ipc/crossTeam.ts#L100
'req.fromMember ?? ''' will use Object's default stringification format ('[object Object]') when stringified
lint (main): src/main/ipc/crossTeam.ts#L99
'req.fromTeam ?? ''' will use Object's default stringification format ('[object Object]') when stringified
lint (main): src/main/ipc/cliInstaller.ts#L45
Array type using 'Array<T>' is forbidden. Use 'T[]' instead
lint (main): src/main/index.ts#L495
Provide a compare function that depends on "String.localeCompare", to reliably sort elements alphabetically
lint (main): src/main/index.ts#L494
Provide a compare function that depends on "String.localeCompare", to reliably sort elements alphabetically
lint (main): src/main/index.ts#L375
Prefer using an optional chain expression instead, as it's more concise and easier to read
lint (renderer): src/renderer/components/chat/items/linkedTool/renderHelpers.tsx#L275
Refactor this function to always return the same type
lint (renderer): src/renderer/components/chat/items/TeammateMessageItem.tsx#L106
't' is already declared in the upper scope on line 88 column 13
lint (renderer): src/renderer/components/chat/items/TeammateMessageItem.tsx#L106
't' is already declared in the upper scope on line 88 column 13
lint (renderer): src/renderer/components/chat/UserChatGroup.tsx#L440
Remove this "!==" check; it will always be true. Did you mean to use "!="?
lint (renderer): src/renderer/components/chat/UserChatGroup.tsx#L409
't' is already declared in the upper scope on line 384 column 11
lint (renderer): src/renderer/components/chat/UserChatGroup.tsx#L409
't' is already declared in the upper scope on line 384 column 11
lint (renderer): src/renderer/components/chat/ChatHistory.tsx#L126
't' is already declared in the upper scope on line 44 column 11
lint (renderer): src/renderer/api/httpClient.ts#L200
Missing return type on function
lint (renderer): src/renderer/analytics/productAnalytics.ts#L638
Provide a compare function to avoid sorting elements alphabetically
lint (renderer): src/renderer/analytics/productAnalytics.ts#L225
Provide a compare function to avoid sorting elements alphabetically