Skip to content

Fix/cleanup oneoff artifacts - #48

Merged
LinMoQC merged 2 commits into
mainfrom
fix/cleanup-oneoff-artifacts
Apr 25, 2026
Merged

Fix/cleanup oneoff artifacts#48
LinMoQC merged 2 commits into
mainfrom
fix/cleanup-oneoff-artifacts

Conversation

@LinMoQC

@LinMoQC LinMoQC commented Apr 25, 2026

Copy link
Copy Markdown
Owner

Summary

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactor / code cleanup
  • CI / tooling

Related issue

Closes #

Changes

How to test

Screenshots (if applicable)

Checklist

  • My code follows the project's coding conventions
  • I have run ./lyra lint and there are no type errors
  • I have added/updated tests for the changed functionality
  • I have updated the documentation if behavior changed
  • The PR title follows Conventional Commits format (feat:, fix:, etc.)
  • I have read the CONTRIBUTING.md

@vercel

vercel Bot commented Apr 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
lyra-note-web Ready Ready Preview, Comment Apr 25, 2026 5:31pm

@coderabbitai

coderabbitai Bot commented Apr 25, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: eccac1e5-6683-40b1-9e8f-9812f42c6046

📥 Commits

Reviewing files that changed from the base of the PR and between 7b2b89a and 54ac5bd.

⛔ Files ignored due to path filters (1)
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/85239377-83ad-478b-b819-f36d894bb9e3.pdf is excluded by !**/*.pdf
📒 Files selected for processing (11)
  • .gitignore
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/3acf8dfe-1723-4896-a9e6-23a547d5737a.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/41b24de4-9d20-48d8-bc3c-9fe8fdeb4934.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/72708d3a-a42e-4a96-9fe2-28081462ed31.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/a9b726fd-1b6f-4444-a297-0dbdb73fdd3c.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/a9e24316-a846-4e3c-8b20-52234859609e.md
  • patch_home.js
  • scripts/create_db_table_design_docx.py
  • scripts/fix_thesis_er_diagrams.py
  • scripts/fix_thesis_er_diagrams_mermaid.py
  • scripts/generate_thesis_pdfs.py
💤 Files with no reviewable changes (10)
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/3acf8dfe-1723-4896-a9e6-23a547d5737a.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/41b24de4-9d20-48d8-bc3c-9fe8fdeb4934.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/72708d3a-a42e-4a96-9fe2-28081462ed31.md
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/a9e24316-a846-4e3c-8b20-52234859609e.md
  • scripts/fix_thesis_er_diagrams.py
  • scripts/fix_thesis_er_diagrams_mermaid.py
  • scripts/generate_thesis_pdfs.py
  • apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/a9b726fd-1b6f-4444-a297-0dbdb73fdd3c.md
  • patch_home.js
  • scripts/create_db_table_design_docx.py

📝 Walkthrough

Summary by CodeRabbit

Chores

  • 扩展了 .gitignore 配置,新增对本地存储、代理内存和临时运行数据目录的忽略规则,改进了项目文件管理
  • 清理并移除了多个内部使用的文档文件和自动化脚本工具

Walkthrough

本次改动删除了多个运行时生成的笔记文件、补丁脚本和文档生成工具脚本,同时扩展 .gitignore 规则以忽略 apps/api/ 目录下的运行时缓存和数据目录。

Changes

Cohort / File(s) Summary
配置与存储清理
.gitignore
扩展忽略规则以覆盖 apps/api/storage/apps/api/data/memory/apps/api/tmp/ 目录。
删除笔记文档
apps/api/storage/notebooks/9a2371fa-8a99-4436-b330-f35bf43c893c/*
移除包含回归测试检查、烟测规范和论文设计文档等运行时生成的笔记文件共 5 个。
删除补丁脚本
patch_home.js
删除对 home-page.tsx 进行正则替换的补丁脚本(140 行),用于修改建议列表和 UI 布局。
删除文档生成脚本
scripts/create_db_table_design_docx.py, scripts/fix_thesis_er_diagrams.py, scripts/fix_thesis_er_diagrams_mermaid.py, scripts/generate_thesis_pdfs.py
移除四个文档生成工具脚本(共 1709 行),包括数据库设计文档生成、ER 图表生成(两种方案)和论文 PDF 生成功能。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed 标题'Fix/cleanup oneoff artifacts'清晰准确地概括了主要变更内容——删除临时性文物和清理仓库。
Description check ✅ Passed PR描述虽未填写具体内容,但标题、提交信息和变更文件内容与清理临时制品相关,说明了PR的意图。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/cleanup-oneoff-artifacts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@LinMoQC
LinMoQC merged commit a588922 into main Apr 25, 2026
7 checks passed
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