Skip to content

feat(packages): add uv-backed consumer sources (1/3) - #4535

Draft
TomCC7 wants to merge 1 commit into
cc/test/package-distribution-acceptancefrom
cc/feat/package-consumer-sources
Draft

TomCC7 wants to merge 1 commit into
cc/test/package-distribution-acceptancefrom
cc/feat/package-consumer-sources

Conversation

@TomCC7

@TomCC7 TomCC7 commented Oct 11, 2026

Copy link
Copy Markdown
Member

Problem

  • External packages need a consumer installation flow that can select a Git subproject without editing the dimOS checkout or introducing a custom resolver.

Solution

  • Add dimos package add backed by uv, with local/editable sources, Git revision selectors, subdirectories and extras.
  • Resolve github:OWNER/REPO#distribution from static project/workspace manifests and preserve ordinary pyproject.toml/uv.lock metadata.
  • Restore authored metadata when uv fails; keep native compilation deferred to module preparation.

Validation

@TomCC7
TomCC7 added this pull request to stack #4466 October 11, 2026 19:57

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

go2 replay realtime (arm64)

Details
Benchmark suite Current: b498107 Previous: 1e7b491 Ratio
peak memory 1606.223 MB 1569.52 MB 1.02
peak threads 356 threads 352 threads 1.01
network (transport) 2730.054 MB 2722.016 MB 1.00
disk write 1.668 MB 1.492 MB 1.12
instructions 115.787 G 115.634 G 1.00

This comment was automatically generated by workflow using github-action-benchmark.

@codecov

codecov Bot commented Oct 11, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
6192 1 6191 86
View the top 1 failed test(s) by shortest run time
dimos.codebase_checks.test_no_init_files::test_no_init_files
Stack Traces | 0.043s run time
def test_no_init_files():
        dimos_dir = DIMOS_PROJECT_ROOT / "dimos"
        init_files = sorted(dimos_dir.rglob("__init__.py"))
        # The root dimos/__init__.py is allowed for the porcelain lazy import.
        init_files = [f for f in init_files if f != dimos_dir / "__init__.py"]
        if init_files:
            listing = "\n".join(f"  - {f.relative_to(dimos_dir)}" for f in init_files)
>           raise AssertionError(
                f"Found __init__.py files in dimos/:\n{listing}\n\n"
                "__init__.py files are not allowed because they lead to unnecessary "
                "extraneous imports. Everything should be imported straight from the "
                "source module."
            )
E           AssertionError: Found __init__.py files in dimos/:
E             - package/__init__.py
E           
E           __init__.py files are not allowed because they lead to unnecessary extraneous imports. Everything should be imported straight from the source module.

dimos_dir  = PosixPath('.../dimos/dimos/dimos')
init_files = [PosixPath('.../dimos/dimos/dimos/package/__init__.py')]
listing    = '  - package/__init__.py'

dimos/codebase_checks/test_no_init_files.py:25: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

This branch was successfully deployed

1 active deployment
cachix — b4981074 Deployed Oct 11, 2026 by TomCC7 via cachix-build-macos #12586
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.

1 participant