Skip to content

Community-code notice on pg mod + moddable prompt defaults to yes#272

Merged
UtkarshBhardwaj007 merged 2 commits into
mainfrom
feat/mod-community-notice
Jun 3, 2026
Merged

Community-code notice on pg mod + moddable prompt defaults to yes#272
UtkarshBhardwaj007 merged 2 commits into
mainfrom
feat/mod-community-notice

Conversation

@UtkarshBhardwaj007
Copy link
Copy Markdown
Member

What

Two small UX changes:

  1. playground mod community-code notice. A light warning-tone callout now appears before any app source touches the user's machine:

    • above the app list in the interactive picker
    • on the setup screen for the direct playground mod <domain> path (which skips the picker but still downloads and runs the app's setup.sh)

    The copy follows the marketplace-standard register (community-published, not reviewed, use at your own risk) and is deliberately specific about the real exposure point: modding runs the app's setup script locally. It is a notice, not a gate; setup.sh still runs automatically.

    ╭──────────────────────────────────────────────────────────────────────────╮
    │ Community Code                                                           │
    │ Apps here are open source, published by the community, and not reviewed. │
    │ Modding downloads the source and runs its setup script on your machine.  │
    │ Review anything you don't trust first and use at your own risk.          │
    ╰──────────────────────────────────────────────────────────────────────────╯
    

    The copy lives in one module (src/commands/mod/communityNotice.ts) so both screens stay identical. The picker's viewport math reserves rows for the box so tall lists don't overflow the terminal.

  2. playground deploy moddable prompt defaults to yes. The cursor on "make this app moddable?" now starts on "yes", so plain Enter opts in. First slice of the moddable-by-default work (Moddable-by-default deploy + guided GitHub login & repo creation flow #200); the graceful handling of a missing GitHub origin is a follow-up.

Verification

  • pnpm format:check, pnpm lint:license, pnpm build, pnpm test (579 tests) all pass
  • Callout rendering visually confirmed via a standalone Ink render

Changeset included.

@UtkarshBhardwaj007 UtkarshBhardwaj007 merged commit 66eb632 into main Jun 3, 2026
19 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

Dev build ready — try this branch:

curl -fsSL https://raw.githubusercontent.com/paritytech/playground-cli/main/install.sh | VERSION=dev/feat/mod-community-notice bash

@UtkarshBhardwaj007 UtkarshBhardwaj007 deleted the feat/mod-community-notice branch June 3, 2026 23:25
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

E2E Test Pass · ✅ PASS

Tag: e2e-ci-pr · Branch: feat/mod-community-notice · Commit: 957627b · Run logs

Cell Result Time
pr-install ✅ PASS 0m42s
pr-deploy-cdm ✅ PASS 4m07s
pr-init-session ✅ PASS 2m19s
pr-mod ✅ PASS 1m16s
pr-preflight ✅ PASS 2m23s
pr-deploy-frontend ✅ PASS 9m13s
pr-deploy-foundry ✅ PASS 0m40s
${{ matrix.cell }} ⏭️ SKIP 0m00s
${{ matrix.cell }} ⏭️ SKIP 0m00s

Sentry traces: view spans for this run

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