-
-
Notifications
You must be signed in to change notification settings - Fork 180
release v0.5.3 #712
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
release v0.5.3 #712
Changes from all commits
9a59a9c
feba4ee
78d5e65
5e9fd0d
2900a81
e6e334c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| # syntax=docker/dockerfile:1 | ||
| FROM oven/bun:debian AS deps | ||
| WORKDIR /app | ||
| COPY package.json bun.lockb* ./ | ||
| RUN bun install --frozen-lockfile | ||
|
|
||
| FROM oven/bun:debian AS builder | ||
| WORKDIR /app | ||
| COPY --from=deps /app/node_modules ./node_modules | ||
| COPY . . | ||
| ENV NEXT_TELEMETRY_DISABLED=1 | ||
| ENV CI=true | ||
| RUN bun run build | ||
|
|
||
| FROM node:20-slim AS runner | ||
| WORKDIR /app | ||
| ENV NODE_ENV=production | ||
| ENV PORT=8080 | ||
| EXPOSE 8080 | ||
|
|
||
| # 关键:确保复制了所有必要的文件,特别是 drizzle 文件夹 | ||
| COPY --from=builder /app/public ./public | ||
| COPY --from=builder /app/.next ./.next | ||
| COPY --from=builder /app/node_modules ./node_modules | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The It's a best practice to only include production dependencies in the final image. You can achieve this by adding a new stage specifically for production dependencies: # After the 'builder' stage
FROM oven/bun:debian AS prod-deps
WORKDIR /app
COPY package.json bun.lockb* ./
RUN bun install --frozen-lockfile --production
# In the 'runner' stage
# ...
COPY --from=prod-deps /app/node_modules ./node_modules
# ...This change will result in a smaller and more secure production image. |
||
| COPY --from=builder /app/package.json ./package.json | ||
| COPY --from=builder /app/drizzle ./drizzle | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. trailing whitespace after Prompt To Fix With AIThis is a comment left during a code review.
Path: Dockerfile
Line: 26:26
Comment:
trailing whitespace after `./drizzle`
How can I resolve this? If you propose a fix, please make it concise. |
||
|
|
||
| CMD ["node", "node_modules/.bin/next", "start"] | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| ALTER TABLE "notification_target_bindings" ALTER COLUMN "schedule_timezone" DROP DEFAULT;--> statement-breakpoint | ||
| ALTER TABLE "providers" ADD COLUMN "anthropic_max_tokens_preference" varchar(20);--> statement-breakpoint | ||
| ALTER TABLE "providers" ADD COLUMN "anthropic_thinking_budget_preference" varchar(20); | ||
|
Comment on lines
+1
to
+3
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. verify this migration includes DROP COLUMN statements for removed Prompt To Fix With AIThis is a comment left during a code review.
Path: drizzle/0060_bored_gertrude_yorkes.sql
Line: 1:3
Comment:
verify this migration includes DROP COLUMN statements for removed `join_claude_pool` and `codex_instructions_strategy` fields - current migration only adds new columns without cleaning up deprecated ones
How can I resolve this? If you propose a fix, please make it concise. |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| ALTER TABLE "system_settings" ADD COLUMN "enable_thinking_budget_rectifier" boolean DEFAULT true NOT NULL; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 627
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 94
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 237
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 98
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 7556
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 237
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 1499
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 2016
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 2149
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 94
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 774
🏁 Script executed:
Repository: ding113/claude-code-hub
Length of output: 721
Dockerfile 端口配置错误:应为 3000 而非 8080
Dockerfile 第 18-19 行设置
PORT=8080与实际部署配置不一致。根据 docker-compose.yaml,容器内部应监听端口 3000(对外通过 APP_PORT 映射为 23000)。健康检查亦使用http://localhost:3000/api/actions/health。建议改为:🤖 Prompt for AI Agents