Skip to content

feat: v0.1.87#1093

Merged
Henry-811 merged 9 commits into
mainfrom
dev/v0.1.87
May 15, 2026
Merged

feat: v0.1.87#1093
Henry-811 merged 9 commits into
mainfrom
dev/v0.1.87

Conversation

@Henry-811
Copy link
Copy Markdown
Collaborator

@Henry-811 Henry-811 commented May 13, 2026

PR Title Format

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

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.

Expected Results

Description/screenshots of expected results.

Additional context

Add any other context about the PR here.

Summary by CodeRabbit

  • Chores

    • Package version bumped to 0.1.87.
  • Documentation

    • Added an llms.txt index and a generated llms-full.txt dump; updated README and docs index with links and an AI-agent note.
    • Added comparative reference pages for AutoGen/AG2, CrewAI, and LangGraph; expanded comparisons section.
  • Bug Fixes

    • Discriminator flow now runs as a single-shot (refinements disabled) to enforce intended behavior.
  • Tests

    • Tests updated to validate the single-shot discriminator behavior.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Note

Reviews paused

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

Adds llms.txt index and automated llms-full.txt generation to the docs, adds three comparison reference pages and TOC changes, enforces discriminator single-shot spawn by passing refine=False (with test assertion), and bumps package version to 0.1.87.

Changes

Documentation: llms index and comparisons

Layer / File(s) Summary
llms index and Sphinx generation
docs/source/_extra/llms.txt, docs/source/conf.py, docs/source/index.rst, README.md, README_PYPI.md
Adds a curated llms.txt, configures html_extra_path and a build-finished hook to emit llms-full.txt, and inserts AI-agent notes in READMEs and index.
Comparisons reference pages
docs/source/reference/comparisons.rst, docs/source/reference/comparisons/autogen.rst, .../crewai.rst, .../langgraph.rst
Replaces small comparisons note with a "More Comparisons" toctree and adds detailed comparison pages for AutoGen/AG2, CrewAI, and LangGraph.

Orchestrator behavior, tests, and version

Layer / File(s) Summary
Discriminator single-shot spawn & test
massgen/orchestrator.py, massgen/tests/test_bootstrap_criteria.py
Calls manager.spawn_subagent(..., refine=False) for single-shot discriminator subagent and asserts refine=False in the related unit test.
Version constant update
massgen/__init__.py
Module-level __version__ constant updated to 0.1.87.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

Possibly related PRs

  • massgen/MassGen#1090: Related change to spawn discriminator subagent with refine=False targeting the same discriminator/subagent flow.
  • massgen/MassGen#1091: Related implementation/testing of bootstrap_subagent discriminator behavior.
  • massgen/MassGen#1087: Related bootstrap_subagent discriminator emergence flow tied to subagent spawning behavior.

Suggested reviewers

  • ncrispino
  • a5507203
🚥 Pre-merge checks | ✅ 3 | ❌ 5

❌ Failed checks (4 warnings, 1 inconclusive)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description consists entirely of the template itself with all sections left unfilled or empty, providing no actual information about the changes, rationale, or testing. Complete the description with the actual brief description of changes, mark the relevant change type(s), fill in the pre-commit status, and provide test commands and expected results.
Docstring Coverage ⚠️ Warning Docstring coverage is 60.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Documentation Updated ⚠️ Warning Two new Python functions in docs/source/conf.py lack required Google-style docstrings per coding guidelines. CHANGELOG.md missing v0.1.87 entry for this release PR. Add Google-style docstrings to _generate_llms_full_txt() and setup() in docs/source/conf.py. Add v0.1.87 section to CHANGELOG.md documenting version bump, llms.txt infrastructure, comparison pages, and refine=False changes.
Config Parameter Sync ⚠️ Warning Parameter upload_files exists in API handler but missing from backend. Configuration parameter sync required. Add "upload_files" to massgen/backend/base.py's get_base_excluded_config_params() to match _api_params_handler_base.py.
Title check ❓ Inconclusive The title 'feat: v0.1.87' is too vague and does not convey meaningful information about the actual changes made in the pull request. Replace with a more descriptive title that summarizes the main changes, such as 'feat: add llms.txt support and comparison pages for v0.1.87'.
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Capabilities Registry Check ✅ Passed Check not applicable: no backend or model changes detected. PR modifies only version, documentation, orchestrator params, and tests.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch dev/v0.1.87

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

ncrispino and others added 5 commits May 15, 2026 07:58
Closes #1083: three new comparison pages under reference/comparisons/
covering MassGen vs CrewAI, LangGraph, and AutoGen/AG2. Updates the
comparisons hub to drop the "coming soon" note and add a toctree.

Closes #1082: publishes llms.txt (curated, llmstxt.org spec) and
llms-full.txt (concatenated docs corpus) at the docs site root via
html_extra_path and a Sphinx build-finished hook in conf.py. README
and index.rst gain one-line pointers for AI agents and crawlers.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…page

Two follow-ups discovered when previewing the docs locally:

- `href="/llms.txt"` resolved to `file:///llms.txt` on local builds (and
  would 404 on RTD without a root redirect). Switched to relative
  `href="llms.txt"` which works in both contexts.
- The "How Does MassGen Compare?" section only mentioned LLM Council.
  Expanded it to list all four comparison pages (LLM Council, CrewAI,
  LangGraph, AutoGen/AG2) with their core differentiators.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
docs: add comparison pages and llms.txt index
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/source/conf.py`:
- Around line 283-285: Add a Google-style docstring to the setup function that
explains its purpose (registering the "build-finished" event to call
_generate_llms_full_txt), documents the parameter app (Sphinx application
object) and the return value (a dict containing "parallel_read_safe" and
"parallel_write_safe" booleans), and any side effects; place the docstring
immediately after def setup(app): and use the Google docstring sections: Args
and Returns, referencing setup and _generate_llms_full_txt so readers know what
event is connected.
- Around line 232-280: Add a Google-style docstring to the
_generate_llms_full_txt function describing its purpose (generate a concatenated
llms-full.txt from docs sources), listing Args (app: Sphinx application,
exception: Exception or None) with types and behavior, explaining Returns (None)
and any side effects (writes out llms-full.txt, prints warnings), and noting the
conditions under which it exits early (non-html builder or non-None exception)
and errors raised/handled (OSError, UnicodeDecodeError). Place the docstring
immediately under the def _generate_llms_full_txt(...) line and follow Google
docstring sections: short summary, Args, Returns, and Raises/Side effects as
appropriate.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ae8f4b7c-e656-480d-a26a-2417b659711d

📥 Commits

Reviewing files that changed from the base of the PR and between a866e9e and b50c34f.

📒 Files selected for processing (11)
  • README.md
  • README_PYPI.md
  • docs/source/_extra/llms.txt
  • docs/source/conf.py
  • docs/source/index.rst
  • docs/source/reference/comparisons.rst
  • docs/source/reference/comparisons/autogen.rst
  • docs/source/reference/comparisons/crewai.rst
  • docs/source/reference/comparisons/langgraph.rst
  • massgen/orchestrator.py
  • massgen/tests/test_bootstrap_criteria.py
✅ Files skipped from review due to trivial changes (6)
  • README.md
  • docs/source/reference/comparisons.rst
  • docs/source/reference/comparisons/crewai.rst
  • docs/source/reference/comparisons/langgraph.rst
  • docs/source/index.rst
  • docs/source/reference/comparisons/autogen.rst

Comment thread docs/source/conf.py
Comment thread docs/source/conf.py
@ncrispino
Copy link
Copy Markdown
Collaborator

@coderabbitai pause

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 15, 2026

✅ Actions performed

Reviews paused.

@Henry-811 Henry-811 merged commit cccc43d into main May 15, 2026
20 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request May 16, 2026
18 tasks
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.

3 participants