Skip to content

feat: add yarn setup:oss for open-source contributors - #53

Merged
MounirDhahri merged 1 commit into
mainfrom
claude/oss-setup
Jul 13, 2026
Merged

feat: add yarn setup:oss for open-source contributors#53
MounirDhahri merged 1 commit into
mainfrom
claude/oss-setup

Conversation

@MounirDhahri

Copy link
Copy Markdown
Member

Description

Open-source contributors currently can't run yarn setup:artsy (it pulls the private Unica77LL fonts + env from Artsy S3), and without fonts expo prebuild fails on the missing assets/fonts/Unica77LL-*.otf. This adds a no-private-access setup path.

scripts/setup-oss (yarn setup:oss):

  • Writes empty placeholder Unica77LL fonts into assets/fonts so expo prebuild / the expo-font plugin resolve them. The app falls back to the system font; drop real .otf files in assets/fonts to use your own typefaces. The font list mirrors the expo-font fonts array in app.json.
  • Creates keys.json from keys.example.json if missing (react-native-keys).
  • Idempotent; needs no AWS/S3.

Also:

  • package.json — new setup:oss script.
  • README — filled in the "For OSS Contributors" section (was "Instructions coming later") and pointed the template steps at setup:oss.
  • docs/getting-started — documented yarn setup:oss alongside setup:artsy.

Verified the script runs and is idempotent (creates the 4 placeholder fonts + keys.json).

PR Checklist

  • No app code touched; setup tooling + docs only.
  • Placeholder-font approach matches what CI already uses for prebuild.

🤖 Generated with Claude Code


Generated by Claude Code

Adds a setup path that needs no Artsy/S3 access, so OSS contributors don't
have to source the private assets/fonts/Unica77LL-*.otf fonts. scripts/setup-oss:
- writes empty placeholder Unica77LL fonts into assets/fonts so expo prebuild /
  the expo-font plugin resolve them (app falls back to system fonts; drop real
  .otf files to override). Font list mirrors app.json.
- creates keys.json from keys.example.json if missing (react-native-keys).
Idempotent. Wires up package.json 'setup:oss' and documents it in the README
(OSS Contributors + template steps) and Getting Started.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01KPWjhmaeq8DnsiiFD9Eg7A
@MounirDhahri
MounirDhahri merged commit d1536f7 into main Jul 13, 2026
5 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