Skip to content

feat: Nlip integration#475

Merged
Henry-811 merged 14 commits intodev/v0.1.13from
nlip_integration
Nov 17, 2025
Merged

feat: Nlip integration#475
Henry-811 merged 14 commits intodev/v0.1.13from
nlip_integration

Conversation

@qidanrui
Copy link
Copy Markdown
Contributor

@qidanrui qidanrui commented Nov 8, 2025

PR Title Format

Your PR title must follow the format: <type>: <brief description>

Closes MAS-54

Valid types:

  • fix: - Bug fixes
  • feat: - New features
  • breaking: - Breaking changes
  • docs: - Documentation updates
  • refactor: - Code refactoring
  • test: - Test additions/modifications
  • chore: - Maintenance tasks
  • perf: - Performance improvements
  • style: - Code style changes
  • ci: - CI/CD configuration changes

Examples:

  • fix: resolve memory leak in data processing
  • feat: add export to CSV functionality
  • breaking: change API response format
  • docs: update installation guide

Description

Brief description of the changes in this PR

Type of change

  • Bug fix (fix:) - Non-breaking change which fixes an issue
  • New feature (feat:) - Non-breaking change which adds functionality
  • Breaking change (breaking:) - Fix or feature that would cause existing functionality to not work as expected
  • Documentation (docs:) - Documentation updates
  • Code refactoring (refactor:) - Code changes that neither fix a bug nor add a feature
  • Tests (test:) - Adding missing tests or correcting existing tests
  • Chore (chore:) - Maintenance tasks, dependency updates, etc.
  • Performance improvement (perf:) - Code changes that improve performance
  • Code style (style:) - Changes that do not affect the meaning of the code (formatting, missing semi-colons, etc.)
  • CI/CD (ci:) - Changes to CI/CD configuration files and scripts

Checklist

  • I have run pre-commit on my changed files and all checks pass
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Pre-commit status

# Paste the output of running pre-commit on your changed files:
# uv run pre-commit install
# git diff --name-only HEAD~1 | xargs uv run pre-commit run --files # for last commit
# git diff --name-only origin/<base branch>...HEAD | xargs uv run pre-commit run --files # for all commits in PR
# git add <your file> # if any fixes were applied
# git commit -m "chore: apply pre-commit fixes"
# git push origin <branch-name>

How to Test

Add test method for this PR.

Test CLI Command

Write down the test bash command. If there is pre-requests, please emphasize.

 uv run python -m massgen.cli --config massgen/configs/examples/nlip_openai_weather_test.yaml "whats the sum of 123 and 456? and whats the weather of Tokyo?"

Expected Results

Description/screenshots of expected results.

Additional context

Add any other context about the PR here.

@qidanrui qidanrui marked this pull request as draft November 8, 2025 02:06
@qidanrui qidanrui added this to the 0.1.13 milestone Nov 12, 2025
…router functionality. Update agent configuration to allow per-agent NLIP settings and improve error handling in NLIP message routing.
…ple backends, enhancing error handling and logging. Fall back to direct execution when NLIP routing fails or is disabled.
…gement. Introduce a new configuration example for OpenAI with NLIP integration
@qidanrui qidanrui marked this pull request as ready for review November 17, 2025 01:01
@Henry-811 Henry-811 changed the base branch from main to dev/v0.1.13 November 17, 2025 15:16
@Henry-811 Henry-811 merged commit 8e06669 into dev/v0.1.13 Nov 17, 2025
3 of 21 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.

4 participants