Skip to content

Apply the OctoSense icon format throughout new-app flows - #196

Merged
guofoo merged 1 commit into
mainfrom
feature/octosense-icon-authoring
Oct 21, 2026
Merged

guofoo merged 1 commit into
mainfrom
feature/octosense-icon-authoring

Conversation

@guofoo

@guofoo guofoo commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

New apps need a distinct icon in the approved OctoSense family, with artwork authored independently of platform masks. Update root/generated-app agent instructions, the script starter SVG and READMEs, quickstart, publishing checklist, card-template pointer and script/image-to-card flows so the same rules reach every new app.

The default format is a 64-point square master with foreground in the central 66%, a restrained illustrated object and ivory/ink materials with supporting accents, and no baked platform corners or outer shadow. The template demonstrates the format; authors must replace its identity before submission and review 64/40/24 px and platform frames. Store apps declare their own bundle artwork, without modifying Makepad or OctoSense's built-in replacement registry.

App Hub's ICONS guide owns packaging and admission limits; it has a matching update. The approved family is tracked in OctoSense PR #555. No admission policy, runtime pins, release workflow or historical evidence changes.

Validation:

  • Onboarding suite: 39 tests run, 34 passed and 5 optional build tests skipped.
  • Relative Markdown links: 0 broken, 0 pending; whitespace checks pass.
  • A fresh tools/octo new invocation copies the exact shared starter SVG and updated AGENTS instructions.
  • SVG parsing, square/full-bleed geometry, size and raster-measured central 66% foreground checks pass; the source matches App Hub's starter byte for byte.
  • The freshly generated script fixture passes icon file checks with a current pinned Hub CLI and has only the expected missing-screenshot listing refusal. No screenshot placeholders were created.

Native UI, devices and optional Wasm build fixtures remain unverified for this change; no app is being published.

@guofoo
guofoo merged commit 703b58c into main Oct 21, 2026
8 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.

1 participant