Skip to content

合并#4077

Open
northeast18 wants to merge 15 commits into
notionnext-org:mainfrom
northeast18:main
Open

合并#4077
northeast18 wants to merge 15 commits into
notionnext-org:mainfrom
northeast18:main

Conversation

@northeast18
Copy link
Copy Markdown

尽量按此模板PR内容,或粘贴相关的ISSUE链接。

已知问题

  1. (示例)版本号管理不规范
    • 版本号直接写在环境变量中,容易出错
    • 多处维护版本号,可能不一致

解决方案

  1. (示例)将版本号管理从 .env.local 迁移到 package.json
    • 统一从 package.json 读取版本号
    • 使用 IIFE 优雅处理版本号获取逻辑
    • 保持向后兼容,支持环境变量覆盖

改动收益

  1. (示例)更规范的版本管理
    • 统一从 package.json 读取
    • 保持与 npm 生态一致
    • 减少人为错误

具体改动

  1. (示例)blog.config.js
    • 移除原有的静态版本号配置
    • 在文件末尾添加动态版本号获取逻辑
    • 保持向后兼容,优先使用环境变量
    • 添加错误处理和默认值

测试确认

  • 本地开发环境测试通过
  • 生产环境构建测试通过
  • (如适用)版本号正确显示
  • (如适用)环境变量配置正常工作

用户文档(docs/user-guide/

若本 PR 修改 docs/user-guide/docs/developer/ 中与站长相关的说明,可勾选「不适用」并跳过本节。

  • 不适用(无文档改动)
  • 已按 维护工作流 自检
  • 路径符合 docs/user-guide/ 目录约定
  • 已更新 user-guide/README.md(新增/移动文章时)
  • 已更新 ARTICLE_INDEX.md(新 slug 或路径变更时)
  • 环境变量名与 conf/*.config.js 一致(若文档涉及配置)
  • 示例中无真实 Token、.env、私有 ID
  • 保留或更新了「原文链接」(若源自 docs.tangly1024.com)

文档说明(可选):对应官方 slug / URL、是否与功能 PR 配套

northeast18 and others added 15 commits December 14, 2025 14:46
Updated the workflow to include a trigger-deploy job and added comments for clarity.
- Add defensive check in getPageTableOfContents to skip invalid blockId
- Fix transclusion_reference handling to check if pointer exists
- Add null check in all theme Catalog/Toc components before calling uuidToId

This fixes the build error:
TypeError: Cannot read properties of undefined (reading 'replaceAll')
    at uuidToId (notion-utils/build/index.js:533:31)
- Add validation for childBlock structure when expanding sync_block
- Ensure childBlock has valid id before adding to clonedBlock
- Filter out blocks without valid id in formatNotionBlock
- Add id validation in adapterNotionBlockMap

This fixes the uuidToId error in react-notion-x Block component.
实现sitemap.xml验证
@northeast18 northeast18 requested a review from tangly1024 as a code owner May 18, 2026 22:42
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