Merge pull request #458 from 777genius/chore/reviewrouter-v1.0.127-main #3816
777geniusci.yml
on: push
validate
1m 7s
Task change ledger Windows smoke
4m 52s
Matrix: lint
Matrix: test
Annotations
1 error and 34 warnings
|
validate
Process completed with exit code 1.
|
|
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
|