Skip to content

接口分享功能增加json输出功能,方便AI调用#2484

Open
350050183 wants to merge 9 commits intostar7th:masterfrom
350050183:master
Open

接口分享功能增加json输出功能,方便AI调用#2484
350050183 wants to merge 9 commits intostar7th:masterfrom
350050183:master

Conversation

@350050183
Copy link
Copy Markdown
Contributor

核心调整内容

  1. 访问首页时无限重定向循环
  2. JSON API 链接访问方法

修改内容

  1. index.php - 路由和预处理层优化
  • 添加预处理层在 Slim 框架初始化前直接处理 /web/ 静态文件
  • 修改安装状态检测逻辑,排除 API 请求(通过检查 $_GET['s'] 参数)
  • 添加通用 API 路由处理器 /api/{path:.*},支持动态调用 API 控制器方法
  • 对 GET 请求自动转换查询参数为请求属性
  1. ShareModal/index.vue - JSON API 链接生成
  • 修改 shareJsonLink 计算属性
  • 改为使用直接路径 /api/page/jsonByKey?unique_key=...
  • 替代之前的 /index.php?s=/api/page/jsonByKey&unique_key=... 方式
  1. IndexController.php - 重定向路径修正
  • 将相对路径 ./web/#/ 改为绝对路径 /web/#/
  • 保持与兜底路由的一致性

…nstraints for carbonphp/carbon-doctrine-types and doctrine/inflector. Added new API endpoint to retrieve page JSON data by unique key. Improved installation status checks in index.php and IndexController.php. Updated content hash in composer.lock.
@star7th
Copy link
Copy Markdown
Owner

star7th commented Mar 3, 2026

改动有点多,显示有几百个文件变更。我没法审查合理性。暂时不接受合并了。

@star7th
Copy link
Copy Markdown
Owner

star7th commented Mar 3, 2026

AI调用这块,其实我也在思考,想着用MCP是否合理。这块没想好

@350050183
Copy link
Copy Markdown
Contributor Author

350050183 commented Mar 3, 2026

有用的修改是这些,其它是因为之前没有忽略server/vendor/里的代码
index.php
server/app/Api/Controller/PageController.php
server/app/Home/Controller/IndexController.php
web/assets/assets-5c7e27f4.js
web/assets/components-83441848.js
web/assets/components-ee5f5034.css
web/assets/core-73629a4b.js
web/assets/index-8b39095e.js
web/assets/views-4574a868.css
web/assets/views-c90ef895.js
web/index.html
web_src/src/i18n/en-US/item.ts
web_src/src/i18n/zh-CN/item.ts

修改功能截图
QQ_1772524754571

cusor里使用
QQ_1772524818308

@star7th
Copy link
Copy Markdown
Owner

star7th commented Mar 3, 2026

感觉需要设计一种优雅的方式让AI读取和写入文档。MCP也好,开放API也好。我需要好好想想设计。就你现在采用的分享json的方式,个人觉得不够优雅

@350050183
Copy link
Copy Markdown
Contributor Author

主要是给AI阅读没有考虑那么多了,哈

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