bug: detectExternalDeps does not strip PEP 508 environment markers or inline comments from requirements.txt / pyproject.toml

[claude]

The detectExternalDeps function in internal/local/local.go does not handle two common Python dep-spec patterns.

1. PEP 508 environment markers (; separator)

A line like requests; python_version >= "3.0" is processed as:

• The separator loop strips >= at the marker position, not the package boundary
• Result added to deps: "requests; python_version" — wrong

For marker-only-< form: Flask; python_version < "3.8" produces "Flask; python_version" — wrong

2. Inline comments

A line like requests # needed for API has no version specifier, so none of the current separators match. The whole string becomes the package name — wrong (pip itself strips inline comments).

Affected locations

• internal/local/local.go:213-227 — requirements.txt parser
• internal/local/local.go:291-302 — pyproject.toml single-line array parser
• internal/local/local.go:315-325 — pyproject.toml multi-line array parser

Missing test coverage

internal/local/local_test.go covers version specifiers, extras, PEP 508 direct URLs, and flags — but has no tests for environment markers or inline comments.

Fix

Prepend ";" and " #" to the separator list at all three parser sites, so everything from those characters onward is stripped. Add test cases for both patterns.

@claude please implement this

[claude]

Claude finished @claude[bot]'s task in 5m 53s —— View job • claude/issue-478-20260302-0539 • [Create PR ➔](https://github.com/supermodeltools/Uncompact/compare/main...claude/issue-478-20260302-0539?quick_pull=1&title=Issue%20%23478%3A%20Changes%20from%20Claude&body=This%20PR%20addresses%20issue%20%23478%0A%0AGenerated%20with%20%5BClaude%20Code%5D(https%3A%2F%2Fclaude.ai%2Fcode)

---

Tasks

• Gather context and read relevant files
• Understand the request and existing code
• Fix separator list at all three parser sites in internal/local/local.go
• Add test cases for environment markers and inline comments in internal/local/local_test.go
• Build and verify (go build ./... and go vet ./... both pass)
• Commit and push changes
• Create PR

Summary

Fixed detectExternalDeps to correctly handle two Python dep-spec patterns that were previously producing wrong package names:

PEP 508 environment markers — requests; python_version >= "3.0" now yields requests instead of "requests; python_version"

Inline comments — flask # needed for API now yields flask instead of the full raw line

Changes:

• internal/local/local.go — prepended ";" and " #" to the separator list at all 3 parser sites (lines 221, 293, 316)
• internal/local/local_test.go — added 4 new test cases: env markers and inline comments for both requirements.txt and pyproject.toml

View PR #483