Skip to content

Conversation

@zmyzxb
Copy link
Collaborator

@zmyzxb zmyzxb commented Nov 26, 2025

Describe this PR

What changed?

Why?

Related issues

Checklist for PR

  • Write a descriptive PR title following the Angular commit message format: <type>(<scope>): <subject>

    • Examples: feat(agent): add pdf tool via mcp, perf: make llm client async, fix(utils): load custom config via importlib
    • Valid types: feat, fix, docs, style, refactor, perf, test, build, ci, revert
    • The check-pr-title CI job will validate your title format
    • Bad title examples and why they fail:
      • Update README ❌ Missing type and colon
      • feat add new feature ❌ Missing colon after type
      • Feature: add new tool ❌ Invalid type (should be feat)
      • feat(Agent): add tool ❌ Scope should be lowercase
      • feat(): add tool ❌ Empty scope not allowed
      • feat(my_scope): add tool ❌ Underscores not allowed in scope
      • feat(my space): add tool ❌ Space not allowed in scope
      • feat(scope):add tool ❌ Missing space after colon
      • feat(scope): ❌ Empty subject
  • Run lint and format locally:

    • uv tool run ruff@0.8.0 check --fix .
    • uv tool run ruff@0.8.0 format .
    • CI job lint enforces ruff default format/lint rules on all new codes.

@zmyzxb zmyzxb changed the title Fixes #97: set default api keys for tool-reading Fixes #97: set default API keys for tool-reading Nov 26, 2025
@zmyzxb zmyzxb changed the title Fixes #97: set default API keys for tool-reading fix(logger: #97 set default API keys for tool-reading Nov 26, 2025
@zmyzxb zmyzxb changed the title fix(logger: #97 set default API keys for tool-reading fix(tool): #97 set default API keys for tool-reading Nov 26, 2025
@zmyzxb zmyzxb merged commit 8a0cc68 into main Nov 26, 2025
5 of 8 checks passed
@zmyzxb zmyzxb deleted the zmy/issue97_fix branch November 26, 2025 15:19
Zhudongsheng75 pushed a commit to open-compass/MiroFlow that referenced this pull request Dec 27, 2025
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