fix(api): allow session PR text lint - #738
Conversation
|
Note Gittensory Gate skippedPR closed before full evaluation. No late first comment was created.
💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers. |
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
gittensory · advisory review Reviewed 2 changed file(s) — two independent AI reviewers. Suggested action: ✅ Safe to merge — both reviewers found no blocking issues. Reviewer A · Suggestions
Worth double-checking
Reviewer B · Suggestions
|
Motivation
POST /v1/lint/pr-text, but ordinary session-authenticated users were receiving403because the path was not allowlisted incanSessionAccessPath, creating an availability regression.Description
LINT_PR_TEXT_PATHconstant for"/v1/lint/pr-text"and include it in thecanSessionAccessPathallowlist so session identities can reach the route.createSessionForGitHubUserand verifies aPOST /v1/lint/pr-textrequest returns200with a deterministicverdict.Testing
npm run typecheck, which completed successfully.npx vitest run test/integration/api.test.ts --config vitest.config.ts --reporter verbose, and the integration test file passed (1 file, 34 tests passed).Codex Task