Skip to content

Optimize github action for service deployment - #3

Open
wx40217 wants to merge 3 commits into
mainfrom
cursor/optimize-github-action-for-service-deployment-be7c
Open

wx40217 wants to merge 3 commits into
mainfrom
cursor/optimize-github-action-for-service-deployment-be7c

Conversation

@wx40217

@wx40217 wx40217 commented Oct 29, 2025

Copy link
Copy Markdown

This pull request contains changes generated by a Cursor Cloud Agent

Open in Cursor Open in Web

Co-authored-by: yps302 <yps302@163.com>
@cursor

cursor Bot commented Oct 29, 2025

Copy link
Copy Markdown

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@wx40217
wx40217 marked this pull request as ready for review October 29, 2025 07:52

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Bug: URL Route Malformed with Incorrect Character

URL route has a typo with misplaced '>' character: "/datasets/uuid:dataset_id/documents/uuid:document_id/segments>" should be "/datasets/uuid:dataset_id/documents/uuid:document_id/segments". This malformed URL will fail to match requests correctly.

internal/router/router.py#L239-L240

https://github.com/dream-stein/llmops-py/blob/26e0c28addad1fa289452dce07b30d522013a575/internal/router/router.py#L239-L240

Fix in Cursor Fix in Web


Bug: Flask Route Registration Error

URL route is missing leading forward slash: "apps/uuid:app_id/published-config" should be "/apps/uuid:app_id/published-config". Without the leading slash, Flask will not properly register this route and requests to /apps/{id}/published-config will not match.

internal/router/router.py#L400-L401

https://github.com/dream-stein/llmops-py/blob/26e0c28addad1fa289452dce07b30d522013a575/internal/router/router.py#L400-L401

Fix in Cursor Fix in Web


cursoragent and others added 2 commits October 29, 2025 08:01
Co-authored-by: yps302 <yps302@163.com>
Co-authored-by: yps302 <yps302@163.com>
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