Add a gea.designWidth manifest field

[skyturkish]

Adds a gea.designWidth manifest field: the logical CSS width an app's stylesheets were authored for. Shells that render at an arbitrary size (phones, resizable desktop windows) divide their real width by it to get the device pixel ratio, so a layout drawn for a small panel scales to fill the surface instead of being magnified or left in a corner. weather declares 273.

Related PRs (merge core first):

• core: Engine fixes for the weather cross-platform example core#60
• docs: https://github.com/geastack/docs/pull/1
• android: Render the weather example correctly on Android android#1
• apple: Render the weather example correctly on macOS and iOS apple#3
• windows: Scale to gea.designWidth and render the weather example correctly on Windows windows#6
• simulator: Inline the app's .env in the web dev server simulator#8
• examples: Make weather a cross-platform example (Android, iOS, macOS, Windows) examples#2

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features
  • App design-width settings now accept either a positive number or a numeric string. When no value is provided, the setting defaults to 0.
• Bug Fixes
  • Invalid design-width values are now reported during app validation, helping surface configuration issues earlier.
  • App summaries now include the normalized design-width value for clearer visibility into the configured setting.

[github-actions]

All contributors have signed the CLA.
_{Posted by the CLA Assistant Lite bot.}

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: de4b7c41-b61a-46f1-8da5-01a5aba3fa04

📥 Commits

Reviewing files that changed from the base of the PR and between 2b3a6cb and 719501e.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 8457bb0f-538a-468a-882d-9b2d11c29260

📥 Commits

Reviewing files that changed from the base of the PR and between 6a37018 and 2b3a6cb.

📒 Files selected for processing (1)

• src/manifest.mjs

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.

---

📝 Walkthrough

Walkthrough

The manifest now normalizes gea.designWidth, validates the normalized value, and includes it in the app summary.

Changes

Design width handling

Layer / File(s)	Summary
Normalize and report design width src/manifest.mjs	normalizeDesignWidth converts absent values to 0, positive finite numbers and numeric strings to numbers, and other values to NaN. normalizeApp stores the normalized value. validateApp reports NaN, and appSummary includes the value.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Feature

Suggested reviewers: dashersw

Merge Risk: ⚪ Minimal · up to 2b3a6

The manifest now accepts and reports the design-width field, with no repository-local behavior regression established; the change appears mergeable under normal checks.

Architecture Summary

Architecture risk: 🔵 Low · up to 2b3a6

The change affects 1 system.

Changed systems: src

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

• observed — src (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

• observed — Modified behavior in src/manifest.mjs: validateApp now adds an error when app.designWidth is NaN.
• observed — Modified behavior in src/manifest.mjs: appSummary now includes the app’s normalized designWidth.
• observed — Modified behavior in src/manifest.mjs: Added exported normalizeDesignWidth: absent values normalize to 0, positive finite numbers and numeric strings to numbers, and other values to NaN.
• observed — Modified behavior in src/manifest.mjs: normalizeApp now sets designWidth from gea.designWidth using normalizeDesignWidth.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 1 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and concisely describes the main change: adding the gea.designWidth manifest field.

✨ Finishing Touches 💡 2

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🛠️ Fix failing CI checks 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

_{Comment @coderabbitai help to get the list of available commands.}

[skyturkish]

recheck