Skip to content

refactor(core_ai): share binary storage byte formatter - #2127

Merged
cursor[bot] merged 2 commits into
mainfrom
cursor/dedupe-storage-bytes-c596
Oct 23, 2026
Merged

cursor[bot] merged 2 commits into
mainfrom
cursor/dedupe-storage-bytes-c596

Conversation

@ucguy4u

@ucguy4u ucguy4u commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add formatBinaryStorageBytes (1024-based B/KB/MB/GB) in core_ai.
  • Replace identical private _formatBytes helpers in model manager diagnostics markdown, AI preferences storage UI, and Mind intelligence home.

Test plan

  • cd packages/core_ai && flutter test test/format/storage_byte_format_test.dart
Open in Web Open in Cursor 

@ucguy4u ucguy4u added the docs-not-needed PR intentionally ships without a docs/wiki update label Oct 23, 2026 — with Cursor
@github-actions

Copy link
Copy Markdown
Contributor

Plugin Module Size Gate

Policy: modules over 3 MB must be delivered as plugins; plugin modules over 5 MB must document cache management.

Module Size Type Status
packages/core_ai 0.59 MB bundled OK
packages/feature_mind 2.95 MB bundled WARN: 1-3 MB requires PR size justification

@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ❌ failure Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@cursor
cursor Bot force-pushed the cursor/dedupe-storage-bytes-c596 branch from 1876f4e to dc8f3f0 Compare October 23, 2026 22:05
cursoragent and others added 2 commits October 23, 2026 22:07
Add formatBinaryStorageBytes and use it from model manager diagnostics,
AI preferences storage UI, and Mind intelligence home.

docs-not-needed: display formatting dedupe only; no behavior change.

Co-authored-by: Uday Chauhan <chauhan.s.uday26@gmail.com>
Co-authored-by: Uday Chauhan <chauhan.s.uday26@gmail.com>
@cursor
cursor Bot force-pushed the cursor/dedupe-storage-bytes-c596 branch from dc8f3f0 to 4252a49 Compare October 23, 2026 22:07
@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ❌ cancelled Analyze, formatting
Core Tests ❌ cancelled Core package unit tests
Airo Mind Shell ❌ cancelled Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ✅ success Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@cursor
cursor Bot merged commit 69f354f into main Oct 23, 2026
20 checks passed
@cursor
cursor Bot deleted the cursor/dedupe-storage-bytes-c596 branch October 23, 2026 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-not-needed PR intentionally ships without a docs/wiki update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants