Skip to content

fix: CLAUDE.md audit warnings — platform coupling + code formatting#43

Merged
Iron-Mark merged 1 commit into
mainfrom
fix/claude-md-audit
May 12, 2026
Merged

fix: CLAUDE.md audit warnings — platform coupling + code formatting#43
Iron-Mark merged 1 commit into
mainfrom
fix/claude-md-audit

Conversation

@JamTheDev
Copy link
Copy Markdown
Contributor

Fixes the two remaining warnings from the CLAUDE.md compliance audit.

1. Remove platformLink from domain entity (ai_model_info.dart)

  • Deleted the platformLink getter and its dart:io import from the domain entity
  • Platform URL resolution is already handled correctly and safely by resolveYoloModelUrl() in the data/presentation layer, which uses kIsWeb + defaultTargetPlatform (safe on web) instead of dart:io Platform (throws on web)
  • platformLink had zero callers — it was dead code

2. dart format compliance (36 files)

  • Ran dart format lib/ test/ to bring all code within the 80-char line limit
  • Fixed two curly_braces_in_flow_control_structures lint warnings in learning_progress_screen.dart introduced by the formatter

All 194 tests pass, flutter analyze clean.

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying kudlit-app with  Cloudflare Pages  Cloudflare Pages

Latest commit: e7cb82b
Status: ✅  Deploy successful!
Preview URL: https://250a4e29.kudlit-app.pages.dev
Branch Preview URL: https://fix-claude-md-audit.kudlit-app.pages.dev

View logs

- ai_model_info: remove platformLink getter and dart:io import;
  platform URL resolution already handled correctly by
  resolveYoloModelUrl() in the data/presentation layer
- learning_progress_screen: add curly braces to if statements
- dart format: apply to all 36 files that were out of compliance
@Iron-Mark Iron-Mark merged commit e4f6a1a into main May 12, 2026
2 checks passed
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.

2 participants