fix(build): add setuptools build dependency for flatdict

[Copilot]

Pull Request

Description

Adds setuptools as an explicit build dependency for flatdict in pyproject.toml via the [tool.uv.extra-build-dependencies] table. When uv builds flatdict==4.0.1 in an isolated environment (such as the devcontainer), it fails with ModuleNotFoundError: No module named 'pkg_resources' because flatdict's setup.py imports pkg_resources without declaring setuptools as a build requirement. This override ensures setuptools is present during the build.

Closes #179

Type of Change

• 🐛 Bug fix (non-breaking change fixing an issue)
• ✨ New feature (non-breaking change adding functionality)
• 💥 Breaking change (fix or feature causing existing functionality to change)
• 📚 Documentation update
• 🏗️ Infrastructure change (Terraform/IaC)
• ♻️ Refactoring (no functional changes)

Component(s) Affected

• deploy/000-prerequisites - Azure subscription setup
• deploy/001-iac - Terraform infrastructure
• deploy/002-setup - OSMO control plane / Helm
• deploy/004-workflow - Training workflows
• src/training - Python training scripts
• docs/ - Documentation

Note: Change is in root pyproject.toml (build configuration), which does not map to an existing component checkbox.

Testing Performed

• Terraform plan reviewed (no unexpected changes)
• Terraform apply tested in dev environment
• Training scripts tested locally with Isaac Sim
• OSMO workflow submitted successfully
• Smoke tests passed (smoke_test_azure.py)

Expected validation: Rebuild the devcontainer from scratch and confirm flatdict installs without pkg_resources errors.

Documentation Impact

• No documentation changes needed
• Documentation updated in this PR
• Documentation issue filed

Checklist

• My code follows the project conventions
• Commit messages follow conventional commit format
• I have performed a self-review
• Documentation impact assessed above
• No new linting warnings introduced

[Copilot]

Pull request overview

This PR fixes a devcontainer build failure where flatdict==4.0.1 requires pkg_resources (provided by setuptools) during its build process but doesn't declare it as a build dependency. The fix adds a uv-specific configuration to ensure setuptools is available when building flatdict in isolation.

Changes:

• Added [tool.uv.extra-build-dependencies] section to pyproject.toml specifying that flatdict requires setuptools as a build dependency

[WilliamBerryiii]

Closing and re-opening this PR to unblock the failed actions permissions issues.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

[WilliamBerryiii]

Thanks for the fix! 🎉 Great catch on the missing setuptools build dependency for flatdict.

I went ahead and updated the PR description to align with our pull request template — added the structured sections (Description, Type of Change, Testing Performed, etc.) and linked this to #179 with a Closes reference so the issue will auto-close on merge. No changes to the code itself, just the PR metadata.

Appreciate the contribution!

[liupeirong]

Actually on a clean Ubuntu 24.04 LTS, opening the dev container for the first time still fails. Now it complains there's no module pkg_resources found. pkg_resources is deprecated, so installing it is more complicated. flatdict doesn't seem to be used anywhere in the repo. @agreaves-ms, can you confirm if it's still needed?

[rezatnoMsirhC]

Actually on a clean Ubuntu 24.04 LTS, opening the dev container for the first time still fails. Now it complains there's no module pkg_resources found. pkg_resources is deprecated, so installing it is more complicated. flatdict doesn't seem to be used anywhere in the repo. @agreaves-ms, can you confirm if it's still needed?

#196 removed flatdict and other unused dependencies, so I agree we should consider if this PR is even needed anymore or if we can close it.