Skip to content

chore: remove fold_subdoc configurations to expose full schema#577

Merged
njzjz merged 2 commits intomasterfrom
copilot/remove-fold-subdoc-configurations
Feb 25, 2026
Merged

chore: remove fold_subdoc configurations to expose full schema#577
njzjz merged 2 commits intomasterfrom
copilot/remove-fold-subdoc-configurations

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 25, 2026

fold_subdoc = True on machine_args and resources_args collapsed sub-documentation into a single line, hiding the full argument schema from AI agents that rely on it for schema discovery.

Changes

  • dpdispatcher/run.py (pep723_args): Removed fold_subdoc = True from machine_args and resources_args
  • dpdispatcher/entrypoints/submit.py (submission_args): Same removals

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • 39.106.xx.xx
    • Triggering command: /home/REDACTED/work/dpdispatcher/dpdispatcher/.venv/bin/python python -m coverage run -p --source=./dpdispatcher -m unittest -v (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>remove fold_subdoc configurations</issue_title>
<issue_description>While fold_subdoc minimizes the documentation, which is friendly to human beings, it prevents AI agents from finding the full schema...</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: njzjz <9496702+njzjz@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove fold_subdoc configurations for schema visibility Remove fold_subdoc configurations to expose full schema Feb 25, 2026
Copilot AI requested a review from njzjz February 25, 2026 08:42
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Feb 25, 2026
@njzjz njzjz marked this pull request as ready for review February 25, 2026 08:44
Copilot AI review requested due to automatic review settings February 25, 2026 08:44
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Feb 25, 2026
@njzjz njzjz changed the title Remove fold_subdoc configurations to expose full schema chore: remove fold_subdoc configurations to expose full schema Feb 25, 2026
@njzjz njzjz enabled auto-merge (squash) February 25, 2026 08:44
@njzjz njzjz disabled auto-merge February 25, 2026 08:44
@njzjz njzjz enabled auto-merge (squash) February 25, 2026 08:44
@dosubot dosubot Bot added the enhancement New feature or request label Feb 25, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 48.35%. Comparing base (1228948) to head (6730a0b).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #577      +/-   ##
==========================================
- Coverage   48.40%   48.35%   -0.06%     
==========================================
  Files          40       40              
  Lines        3962     3958       -4     
==========================================
- Hits         1918     1914       -4     
  Misses       2044     2044              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes fold_subdoc = True from the machine_args and resources_args argument schemas so that downstream schema/documentation consumers (including AI agents doing schema discovery) can see the full nested schema rather than a folded one-line summary.

Changes:

  • Removed fold_subdoc = True from machine_args and resources_args in the PEP 723 metadata argument definition.
  • Removed fold_subdoc = True from machine_args and resources_args in the submission JSON argument definition.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
dpdispatcher/run.py Exposes full nested schema for machine/resources in pep723_args() by removing subdoc folding.
dpdispatcher/entrypoints/submit.py Exposes full nested schema for machine/resources in submission_args() by removing subdoc folding.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@njzjz njzjz merged commit 202ebfd into master Feb 25, 2026
40 checks passed
@njzjz njzjz deleted the copilot/remove-fold-subdoc-configurations branch February 25, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

remove fold_subdoc configurations

3 participants