fix manifest

[paul-nechifor]

No description provided.

[chatgpt-codex-connector]

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

[greptile-apps]

Greptile Summary

This PR improves Python package distribution configuration by excluding fixture directories and development files from the package, while ensuring necessary data files are included:

MANIFEST.in changes:

• Added exclusions for test fixture directories (dimos/agents/fixtures, dimos/mapping/google_maps/fixtures)
• Added exclusion for dimos/web/dimos_interface directory
• Removed the generic prune tests directive (replaced with more specific exclusions)
• Issue: Contains duplicate exclusion for dimos/web/command-center-extension (line 6 uses recursive-exclude, line 21 uses prune)

pyproject.toml changes:

• Added package-data for *.tcss files in dimos.utils.cli (used for Textual UI styling)
• Added package-data for *.txt files in dimos.rxpy_backpressure (includes LICENSE.txt)
• Removed package-data for dimos.web.dimos_interface (consistent with MANIFEST.in exclusion)
• Added readme = "README.md" field to project metadata for better PyPI display

These changes ensure the Python package only includes necessary runtime files while excluding test fixtures and development artifacts.

Confidence Score: 4/5

• This PR is safe to merge with minimal risk - it only modifies packaging configuration
• The changes correctly improve package distribution by excluding test fixtures and development files while including necessary data files. The only issue is a minor duplicate exclusion rule in MANIFEST.in that doesn't affect functionality but should be cleaned up for maintainability
• MANIFEST.in has a duplicate exclusion rule that should be cleaned up (see inline comment)

Important Files Changed

Filename	Overview
MANIFEST.in	Added exclusions for fixture directories and dimos_interface, but contains duplicate exclusion for command-center-extension (lines 6 and 21)
pyproject.toml	Added package-data for .tcss and .txt files, removed dimos_interface package-data, and added readme field - all changes are correct

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant Build as Build System (setuptools)
    participant MANIFEST as MANIFEST.in
    participant Pyproject as pyproject.toml
    participant Package as Python Package

    Dev->>Build: python -m build
    Build->>MANIFEST: Read file exclusion rules
    MANIFEST-->>Build: Exclude fixtures, node_modules, dev files
    Build->>Pyproject: Read package configuration
    Pyproject-->>Build: Include package-data (*.tcss, *.txt, etc.)
    Build->>Build: Filter files based on rules
    Note over Build: Excludes: dimos/agents/fixtures,<br/>dimos/mapping/google_maps/fixtures,<br/>dimos/web/dimos_interface,<br/>command-center-extension (duplicate rule)
    Note over Build: Includes: *.tcss from cli,<br/>*.txt from rxpy_backpressure,<br/>README.md for PyPI
    Build->>Package: Generate distribution package
    Package-->>Dev: .whl/.tar.gz with filtered files

Loading

[greptile-apps]

_{2 files reviewed, 1 comment}

_{Edit Code Review Agent Settings | Greptile}

[greptile-apps]

style: Duplicate exclusion: dimos/web/command-center-extension is excluded on both line 6 (using recursive-exclude) and line 21 (using prune). Remove one.

Suggested change

	# Exclude web development directories
	recursive-exclude dimos/web/command-center-extension *
	recursive-exclude dimos/web/websocket_vis/node_modules *
	recursive-exclude dimos/agents/fixtures *
	recursive-exclude dimos/mapping/google_maps/fixtures *
	recursive-exclude dimos/web/dimos_interface *
	# Exclude development files
	exclude .gitignore
	exclude .gitattributes
	prune .git
	prune .github
	prune .mypy_cache
	prune .pytest_cache
	prune .ruff_cache
	prune .vscode