Skip to content

fix: preserve callable default None providers - #687

Open
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/dead-function-callable-default-provider
Open

fix: preserve callable default None providers#687
ApexWorm wants to merge 1 commit into
peteromallet:mainfrom
ApexWorm:fix/dead-function-callable-default-provider

Conversation

@ApexWorm

@ApexWorm ApexWorm commented Aug 1, 2026

Copy link
Copy Markdown

Problem

The dead-function detector reported intentional fail-closed providers whose literal return None value is used as a typed Callable dataclass default. This incorrectly flags the Frequency Coordination System no_current_tenant_id dependency provider despite active use.

Fix

Pre-index exact local and project-relative imported providers referenced by top-level typed dataclass callable defaults. Exempt only their literal return None bodies; pass, bare return, import-only, and general return-none functions remain findings.

Verification

  • Ruff check on the changed tool files
  • Focused smell tests: 60 passed
  • Full Python smell suite: 284 passed
  • Live project detector check: no dead-function match for access_contracts.py

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