Skip to content

Dev-merge#8

Open
Enimdisy wants to merge 9 commits intomainfrom
dev
Open

Dev-merge#8
Enimdisy wants to merge 9 commits intomainfrom
dev

Conversation

@Enimdisy
Copy link
Copy Markdown
Collaborator

Summary / 概述

  • Merged dev (2026-04-27) and latest main (2026-04-28) into a new integration branch, resolving conflicts while preserving both feature sets.
  • Fixed settings/API issues introduced during merge:
    • Removed duplicated kwargs in backend settings response that caused update failures.
    • Removed duplicated default/log keys in settings service to stabilize config persistence.
  • Fixed frontend route/API compatibility under /nanadraw:
    • Added route compatibility for /nanadraw, /nanadraw/projects, /nanadraw/draw.
    • Added API fallback logic (/nanadraw/api/... -> /api/...) for settings and PDF parsing calls.
  • Fixed MinerU PDF parsing reliability:
    • Root-caused PDF 解析失败: ConnectError to httpx env proxy inheritance on result-zip download stage.
    • Updated MinerU client to use trust_env=False and improved network/timeout error messages.
  • Added PDF UX improvement:
    • Added Select All / 全选 for parsed PDF text so users can quote the full content in one click.
    • Added bilingual i18n strings for the new button.

Type / 类型

  • Bug fix / 修复问题
  • New feature / 新功能
  • Enhancement / 增强
  • Refactoring / 重构
  • Documentation / 文档

Test Plan / 测试计划

  • Backend: cd backend && python -c "from app.main import app"
  • Frontend: cd frontend && npx tsc --noEmit (equivalent run: corepack pnpm type-check)
  • Full test: python start.py and verify UI

Additional validation performed:

  • Backend syntax check: python3 -m py_compile ...
  • Frontend unit test: cd frontend && corepack pnpm test -- src/services/api.test.ts
  • Manual API verification:
    • PUT /api/v1/settings/llm-config returns success
    • POST /api/v1/documents/parse-pdf returns 200 with parsed markdown after MinerU fix

Checklist

  • Code follows project conventions
  • Comments are in English
  • UI text is bilingual (zh/en)
  • No hardcoded secrets

@gemini-code-assist
Copy link
Copy Markdown

Important

Installation incomplete: to start using Gemini Code Assist, please ask the organization owner(s) to visit the Gemini Code Assist Admin Console and sign the Terms of Services.

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.

1 participant