cowork: namespace & side-effect imports consume a module's whole export surface

[Coding-Dev-Tools]

Problem

The dead-code scanner ignored two common import forms entirely:

• import * as Utils from './utils' (namespace import)
• import './polyfill' (bare side-effect import)

Exports consumed only through these forms were falsely reported as unused_export with removable=True — meaning deadcode remove would blank live code. Reproduced pre-fix: a module whose only consumer used Utils.helper() was flagged removable.

Fix

Both forms now resolve their module specifier like barrel star-reexports already do (_resolve_relative_module): when the target resolves to a scanned file, its entire export surface is treated as used. Bare package specifiers (e.g. 'lodash') stay unresolvable and keep flagging local modules correctly.

Tests

New tests/test_namespace_sideeffect_imports.py (5 cases):

1. namespace import marks all target exports used
2. export * as ns from ... re-export ditto
3. bare-specifier namespace import cannot mark local module used
4. side-effect import marks all target exports used
5. control: no consumer still flags exports

Full suite: 121 passed, ruff clean.

[github-actions]

🤖 Automated Code Review

✅ Ruff Lint — No issues

⚠️ Ruff Format — Formatting needed

unformatted: File would be reformatted
 --> src/deadcode/__main__.py:2:1
  |
1 | """Allow running deadcode as: python -m deadcode"""
2 +
3 | from .cli import cli
  |

unformatted: File would be reformatted
   --> src/deadcode/cli.py:66:15
    |
65  | @click.option("--project", "-p", default=".", help="Project directory to scan")
66  + @click.option("--ignore", "-i", multiple=True, help="Additional ignore patterns (gitignore-style)")
67  | @click.option(
    -     "--ignore", "-i", multiple=True, help="Additional ignore patterns (gitignore-style)"
    - )
    - @click.option(
68  |     "--include",
--------------------------------------------------------------------------------
73  | @click.pass_context
    - def cli(
    -     ctx: click.Context, project: str, ignore: tuple[str, ...], include: tuple[str, ...]
    - ) -> None:
74  + def cli(ctx: click.Context, project: str, ignore: tuple[str, ...], include: tuple[str, ...]) -> None:
75  |     """DeadCode — Find and remove dead code in TS/React/Next.js projects.
--------------------------------------------------------------------------------
112 | @cli.command()
    - @click.option(
    -     "--json-output", "-j", is_flag=True, help="Alias for --format=json (deprecated)"
    - )

✅ Secret Detection — Clean

✅ Large Files — Within limits

📊 Diff Stats — 2 file(s) changed

 src/deadcode/scanner.py                    | 32 ++++++++++--
 tests/test_namespace_sideeffect_imports.py | 80 ++++++++++++++++++++++++++++++
 2 files changed, 108 insertions(+), 4 deletions(-)

---

Verdict: ⚠️ Warnings — Lint/format issues found. Recommend fixing before merge.

Automated by Coding-Dev-Tools/.github reusable workflow.