← Back to issue list

Refactor to reduce cyclomatic complexity

View original Github issue

Metadata

Project
fetch-service
Number
#292
Type
issue
State
closed
Author
cmatsuoka
Labels
—
Created
Updated
Closed

Current evaluation

Refactored inspector code to reduce cyclomatic complexity in merged PR #302 (commit 6437810), which fixes #292.

Resolution: closed resolved

Reason: PR #302 was merged as commit 6437810 and explicitly fixes #292. The change extracts common inspector calls and moves artifact operations into the Artefact struct.

Evaluated by: mr-cal-bot

No scores available.

Issue body

### What needs to get done TiCS reports show high cyclomatic complexity in certain subsystems of the Fetch Service. Investigate and refactor to improve this metric. ### Why it needs to get done To make it more maintenable and improve quality metrics of the Fetch Service.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Refactored inspector code to reduce cyclomatic complexity in merged PR #302 (commit 6437810), which fixes #292.
stealth/space-bunny-alpha — closed resolved Complexity refactor merged in canonical/fetch-service#302 (commit 6437810), extracting common inspector calls and moving artifact operations to the Artefact struct.
stealth/space-bunny-alpha — closed resolved Refactoring reduced inspector cyclomatic complexity in merged PR #302, resolving #292. Common calls were extracted and artifact operations moved to the Artefact struct.
qwen/qwen3.6-35b-a3b — — Refactoring the Fetch Service to reduce cyclomatic complexity was closed without implementation. An automated bot linked it to internal Jira ticket CRAFT-3692. No code changes or merges occurred, indicating the task was abandoned or handled offline.

Update history

Date Change
created

Related work

  • Likely Fixed By: fetch-service#302 (confidence 100%)

    Merged complexity refactor; commit 6437810 explicitly fixes #292.

Related issues

Issue Project State Summary Similarity
#496 refactor: Reduce cyclomatic complexity fetch-service merged Merged PR #496 as commit 98cad3a, extracting inspector logic into dedicated functions across four files to reduce cyclomatic complexity.
90%
#302 refactor(inspectors): reduce cyclomatic complexity fetch-service merged Merged as commit 6437810: reduced inspector complexity, moved artifact operations into Artefact, and made Git protocol decoding errors reject artifacts; fixes #292.
79%