Skip to content
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

Feat aisummary wordcount #3091

Merged
merged 8 commits into from
Jan 1, 2025

Conversation

qixing-jk
Copy link
Contributor

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

已知问题

  1. word count和AI summary是DOM通过外部JS实现的
  2. 性能较差,且服务器计算对于AI部分的可扩展性和私密性较好

解决方案

  1. 统一文章静态生成时文章数据处理函数以便统一修改
  2. 静态生成阶段计算word count和AI summary数据

改动收益

  1. 更好的功能体验和SEO,不需要前端二次计算,更好的客户端性能表现

未实现部分和问题

  1. 增加AI summary,导致数据传输倒是有所增加
    但文章界面数据大头是blockmap文章内容,此类增加量不算大,并且可以考虑使用ai summary直接替代原始summary(暂未实现)
  2. 只适配了heo主题
    其他主题一个个适配比较麻烦,主要是我自己用不上
  3. 由于git水平的不足,没有采取功能分支的开发方式和git commit单次合并提交了多个更改导致提交耦合,之后一定注意

具体改动

  1. 详见 files change

测试确认

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

Copy link

vercel bot commented Dec 28, 2024

@qixing-jk is attempting to deploy a commit to the tangly1024's projects Team on Vercel.

A member of the Team first needs to authorize it.

@tangly1024 tangly1024 merged commit d71cf3e into tangly1024:main Jan 1, 2025
3 of 6 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.

2 participants