Skip to content

Deduplicate constant and extract function#1165

Merged
chrarnoldus merged 3 commits intomainfrom
christiaan/constants
Mar 17, 2026
Merged

Deduplicate constant and extract function#1165
chrarnoldus merged 3 commits intomainfrom
christiaan/constants

Conversation

@chrarnoldus
Copy link
Contributor

No description provided.

@kilo-code-bot
Copy link
Contributor

kilo-code-bot bot commented Mar 17, 2026

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
src/app/api/openrouter/[...path]/route.ts 240 Session-authenticated Kilo org members still fail the new Responses API gate because organizationId is only populated for Authorization-header auth.
Other Observations (not in diff)

None.

Files Reviewed (6 files)
  • src/app/api/openrouter/[...path]/route.ts - 1 issue
  • src/lib/handleRequestLogging.ts - 0 issues
  • src/lib/isKiloAffiliatedUser.ts - 0 issues
  • src/lib/processUsage.constants.ts - 0 issues
  • src/lib/processUsage.responses.ts - 0 issues
  • src/lib/processUsage.ts - 0 issues

Fix these issues in Kilo Cloud


Reviewed by gpt-5.4-20260305 · 373,866 tokens

@chrarnoldus chrarnoldus merged commit 84271b5 into main Mar 17, 2026
17 of 18 checks passed
@chrarnoldus chrarnoldus deleted the christiaan/constants branch March 17, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants