Conversation
Test & Lint Summary
Notes:
Prisma schema loaded from prisma/schema.prisma ✔ Generated Prisma Client (v6.18.0) to ./../../node_modules/.pnpm/@prisma+client@6.18.0_prisma@6.18.0_typescript@5.8.3__typescript@5.8.3/node_modules/@prisma/client in 239ms Start by importing your Prisma Client (See: https://pris.ly/d/importing-client) Tip: Need your database queries to be 1000x faster? Accelerate offers you that and more: https://pris.ly/tip-2-accelerate still fails because of pre-existing unsafe assignment errors.
RUN v3.2.4 /workspace/platform/packages/platform-server stdout | tests/slack.config.schemas.test.ts > Slack static config schemas > SlackTriggerStaticConfigSchema: requires app_token and bot_token reference fields ✓ tests/slack.config.schemas.test.ts (2 tests) 1466ms {"level":50,"time":1772277962954,"pid":37214,"hostname":"6216ec4a0d4e","req":{"id":"req-2","method":"GET","url":"/api/containers","query":{},"headers":{"host":"127.0.0.1:43363","connection":"keep-alive","accept":"/","accept-language":"*","sec-fetch-mode":"cors","user-agent":"undici","accept-encoding":"gzip, deflate"},"remoteAddress":"127.0.0.1","remotePort":40734},"res":{"statusCode":503,"headers":{}},"err":{"type":"Error","message":"failed with status code 503","stack":"Error: failed with status code 503\n at onResFinished (/workspace/platform/node_modules/.pnpm/pino-http@11.0.0/node_modules/pino-http/logger.js:115:39)\n at ServerResponse.onResponseComplete (/workspace/platform/node_modules/.pnpm/pino-http@11.0.0/node_modules/pino-http/logger.js:178:14)\n at ServerResponse.emit (node:events:536:35)\n at onFinish (node:_http_outgoing:1031:10)\n at callback (node:internal/streams/writable:766:21)\n at afterWrite (node:internal/streams/writable:710:5)\n at afterWriteTick (node:internal/streams/writable:696:10)\n at processTicksAndRejections (node:internal/process/task_queues:81:21)"},"responseTime":1,"msg":"request errored"} �[32m[Nest] 37237 - �[39m02/28/2026, 11:26:05 AM �[32m LOG�[39m �[38;5;3m[NestFactory] �[39m�[32mStarting Nest application...�[39m |
Test & Lint Summary
Notes:
|
noa-lucent
left a comment
There was a problem hiding this comment.
Requesting changes: found a major configuration-invariant issue in ConfigService that should fail loudly instead of silently defaulting on invalid provider values.
Test & Lint Summary (update)
|
noa-lucent
left a comment
There was a problem hiding this comment.
Re-review complete. The previously requested invariant fix is in place: unsupported LLM_PROVIDER values now fail fast, tests cover the invalid-provider path, and docs were updated to match behavior.
Summary
Testing
Notes
pnpm --filter @agyn/platform-server lintstill fails because of pre-existingrunnerGrpc.clientunsafe assignment errors.pnpm --filter @agyn/platform-server testrequires live Postgres/Docker runner; targeted suites above cover the modified areas.Resolves #1356