Skip to content

fix: resolve MCP tools and Product config problems, optimize higress gateway URI parsing and resource identification#128

Merged
slievrly merged 15 commits intohigress-group:mainfrom
learnerjohn:himarket_product
Dec 18, 2025
Merged

fix: resolve MCP tools and Product config problems, optimize higress gateway URI parsing and resource identification#128
slievrly merged 15 commits intohigress-group:mainfrom
learnerjohn:himarket_product

Conversation

@learnerjohn
Copy link
Copy Markdown
Collaborator

@learnerjohn learnerjohn commented Dec 17, 2025 •
edited
Loading

Description

This PR fixes several bugs and optimizes MCP and Product related functionality:

  • Fix missing tools in direct proxy type MCP configuration
  • Fix inconsistency between Product and API configuration with on-demand sync mechanism
  • Fix missing port and protocol in Higress gateway address parsing
  • Add resource identifiers (subscriptionId, publicationId) for better resource management

Related Issues

Checklist

  • Code has been formatted with mvn spotless:apply
  • Code is self-reviewed
  • Tests added/updated (if applicable)
  • Documentation updated (if applicable)
  • No breaking changes (or migration guide provided)

@learnerjohn
refactor: rename project package to himarket
f2cd523 
Change-Id: I961a47e80c0b30ba94befcf786cd68eb922bd171
@learnerjohn
feat: support Higress gateway IP for Model and MCP
190a3af 
Change-Id: I189047b9adf7e5870413654d49706a914af1ec1b
@learnerjohn
fix: resolve issue where Model cannot use MCP in Higress
834cbfd 
Change-Id: I818a548d4239b13ad7cd57d1e100398b100c1757
@learnerjohn
docs: add user guide and update README with PR guidelines
570869e 
Change-Id: I4c648e944313a85e4da1c10fdc8d477288a10e4c
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
750848d
@learnerjohn
fix: developer consumer config and docs update
9e6c5be 
- Auto-configure Primary Consumer for external developers
- Fix Higress MCP Path
- Remove emoji from documentation files

Change-Id: I02c85054cba5c9459c523d8d964f148d99395d7f
@learnerjohn
style: format code with spotless
061ad1e 
Change-Id: I91e7a57d42ae0e3455dee0ce690e3300d22d6a51
@learnerjohn
merge
96e2a61 
Change-Id: Ieb4d9b1d507301689aefd3f41b9730073705d288
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
8497489
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
1f17d1d
@learnerjohn
fix: correct branding to HiMarket and fix gateway address port handli…
07c092b 
…ng and consumer apikey authentication

Change-Id: I6be6853e5a2819fa46b292d1810fec03294bad87
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_202512
8c5ed3c
@learnerjohn
fix: resolve MCP tools and Product config problems, optimize higress …
ad2caf6 
…gateway URI parsing and resource identification

Change-Id: Ib95eef61fb18fb837f32ab3facdea28e3aa90a7c
@learnerjohn
merge
08447a1 
Change-Id: Ia4d77996cb15ce556529ea1ff7118cb70ae3ecb3
@learnerjohn
Merge remote-tracking branch 'upstream/main' into himarket_product
dbf19b8
@learnerjohn learnerjohn changed the title Himarket product fix: resolve MCP tools and Product config problems, optimize higress gateway URI parsing and resource identification Dec 17, 2025
slievrly
slievrly approved these changes Dec 18, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@slievrly slievrly left a comment

Choose a reason for hiding this comment

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

LGTM

@slievrly slievrly merged commit 33fb004 into higress-group:main Dec 18, 2025
10 of 16 checks passed
@xuechaos xuechaos mentioned this pull request Dec 19, 2025
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@slievrly slievrly slievrly approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

2 participants

@learnerjohn @slievrly