-
-
Notifications
You must be signed in to change notification settings - Fork 0
feat: comprehensive project audit, design system, and product improvements #194
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 171 commits
9a48ab2
3d49703
b6c0d96
403642e
628b16b
a9fde10
26401e3
db02671
535d53c
4055595
305bcb5
fcd26b9
a88f19d
488359e
f34ab5e
4d2c4ac
fff8cd8
0782ea8
1375fcd
51e8112
2e60230
26f2757
043148c
e8250b8
c913465
26bccb5
31a880e
63610e4
73f9938
cd01862
e1eb092
cb9a82e
aa2bbf6
46ae6de
fbb5448
4a087e7
3902345
47fec79
614b26f
06a147d
f5f8f25
5fe153e
56ba2a5
1f61ed7
bf58f4a
7bda4c1
b44fc75
9e16eec
78a20cd
3d16297
d8f8ca6
589fe17
9f6f229
48195ab
8ea9877
016a524
0cd1214
308c229
d1c3cb7
2cb58bd
fcc9d96
9f137d3
36e1a5b
d104dd1
a6b3558
0e435e2
080cb68
3646cb2
57d4303
8153c78
fc9ab96
99b3f56
f4e904c
1e1a44c
5af6419
71bf473
249a2ad
801a738
17c9ff2
3dabece
38a5170
c099dbd
b8a4e81
e013638
c87de4e
2c753b3
6948a26
ff0e19b
35c346e
35f3804
09cf396
d773955
bc424cb
c8d8833
d8846d9
188c7d8
d86c2ec
d096658
65abbd6
d6783af
df64ff5
f9766d3
452f3bd
3f7b49d
5d112ff
3d41f25
5e3af25
4b08f07
7344281
b429ab3
3ed08b4
cdd2db0
177091d
9f2ba75
cbe39c8
38b8128
592d889
09ac09a
5cd5520
2ef5a9a
a8bf258
0999107
40c5216
d65eb5c
8dc7079
024e46c
193bbce
d5cd8ed
cfe513d
21be0a5
b959a25
00e25bf
04e4e18
21af479
4ccab57
e908280
8a1f5c8
a24a231
64d7010
f686ab6
9d64ea9
a472580
6e05279
b2f65aa
01b51cb
f6ddcad
41c811c
078c8f4
7030149
ad325d8
1680e01
31c5474
7fd0ee1
2bf9c1e
20e101c
e80caf8
e3b2ef8
1e20308
fb0dafc
3b01d4b
3f0bc7f
578ad70
ab4a486
dc5bc73
075995d
0e85b24
472bc4e
4e36788
d2d763f
52a2bbe
68c56b6
2672cdc
4ace564
9bccc62
a1f8422
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| # Readied — Environment Variables | ||
| # Copy this file to .env and fill in the values. | ||
| # The desktop app runs fully offline — these are only needed for the API and web app. | ||
|
|
||
| # ─── API (Cloudflare Workers) ─────────────────────────────── | ||
| # Set via `wrangler secret put <KEY>` for deployed environments. | ||
| # For local dev, create packages/api/.dev.vars with these values. | ||
|
|
||
| TURSO_DATABASE_URL=libsql://your-db.turso.io | ||
| TURSO_AUTH_TOKEN= | ||
| JWT_SECRET= # openssl rand -base64 32 | ||
| RESEND_API_KEY= # Resend email service | ||
| SITE_URL=https://readied.app | ||
|
|
||
| # ─── Stripe ───────────────────────────────────────────────── | ||
| STRIPE_SECRET_KEY= | ||
| STRIPE_WEBHOOK_SECRET= | ||
| STRIPE_PRICE_MONTHLY= | ||
| STRIPE_PRICE_ANNUAL= | ||
|
|
||
| # ─── Admin ────────────────────────────────────────────────── | ||
| ADMIN_TOKEN= # Token for /admin endpoints | ||
|
|
||
| # ─── GitHub Actions (repo secrets) ────────────────────────── | ||
| # GH_TOKEN # PAT with repo scope (releases, PRs) | ||
| # CSC_LINK # macOS code signing cert (base64) | ||
| # CSC_KEY_PASSWORD # macOS cert password | ||
| # APPLE_ID # Apple notarization | ||
| # APPLE_APP_SPECIFIC_PASSWORD # Apple app-specific password | ||
| # APPLE_TEAM_ID # Apple Developer Team ID | ||
This file was deleted.
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -37,7 +37,7 @@ jobs: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| fetch-depth: 0 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Setup pnpm | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: pnpm/action-setup@v4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: pnpm/action-setup@v5 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Setup Node.js | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| uses: actions/setup-node@v4 | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
@@ -126,9 +126,9 @@ jobs: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Publish GitHub Release | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| GH_TOKEN: ${{ secrets.GH_TOKEN }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| TAG_NAME: ${{ github.ref_name }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: gh release edit "$TAG_NAME" --draft=false --repo "${{ github.repository }}" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: gh release edit "$TAG_NAME" --draft=false --repo "$GITHUB_REPOSITORY" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| tweet: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| needs: publish | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
@@ -159,11 +159,12 @@ jobs: | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| steps: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| - name: Create sync PR | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| env: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| GH_TOKEN: ${{ secrets.GH_TOKEN }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| TAG_NAME: ${{ github.ref_name }} | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| run: | | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| gh pr create \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --base develop \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --head main \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --title "chore: sync release ${{ github.ref_name }} back to develop" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --title "chore: sync release $TAG_NAME back to develop" \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --body "Auto sync of release commit and changelog." \ | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --repo "${{ github.repository }}" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| --repo "$GITHUB_REPOSITORY" || echo "PR already exists or cannot be created" | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
160
to
+170
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: #!/bin/bash
# Description: Print the sync-develop job and fail while gh pr create still has broad `|| echo` suppression.
python - <<'PY'
from pathlib import Path
text = Path(".github/workflows/build.yml").read_text()
start = text.index(" sync-develop:")
block = text[start:]
print(block)
if "gh pr create" in block and "|| echo" in block:
raise SystemExit("sync-develop still suppresses all gh pr create failures")
PYRepository: tomymaritano/readide Length of output: 659 Do not swallow sync PR creation failures with The broad error suppression on line 170 turns authentication, API, rate-limit, and missing branch errors into silent successes, leaving 🔧 Proposed fix run: |
- gh pr create \
- --base develop \
- --head main \
- --title "chore: sync release $TAG_NAME back to develop" \
- --body "Auto sync of release commit and changelog." \
- --repo "$GITHUB_REPOSITORY" || echo "PR already exists or cannot be created"
+ existing_pr="$(gh pr list \
+ --base develop \
+ --head main \
+ --state open \
+ --repo "$GITHUB_REPOSITORY" \
+ --json number \
+ --jq '.[0].number // empty')"
+
+ if [ -n "$existing_pr" ]; then
+ echo "Sync PR already exists: #$existing_pr"
+ else
+ gh pr create \
+ --base develop \
+ --head main \
+ --title "chore: sync release $TAG_NAME back to develop" \
+ --body "Auto sync of release commit and changelog." \
+ --repo "$GITHUB_REPOSITORY"
+ fi📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1 +1 @@ | ||
| pnpm test && pnpm typecheck | ||
| pnpm typecheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Keep empty env placeholders parser-safe.
The padded inline comments make these values lint failures and can be parsed inconsistently by dotenv tooling. Move the notes above the keys and leave empty placeholders as
KEY=.🧹 Proposed fix
Also applies to: 22-22
🧰 Tools
🪛 dotenv-linter (4.0.0)
[warning] 11-11: [SpaceCharacter] The line has spaces around equal sign
(SpaceCharacter)
[warning] 11-11: [UnorderedKey] The JWT_SECRET key should go before the TURSO_AUTH_TOKEN key
(UnorderedKey)
[warning] 11-11: [ValueWithoutQuotes] This value needs to be surrounded in quotes
(ValueWithoutQuotes)
[warning] 12-12: [SpaceCharacter] The line has spaces around equal sign
(SpaceCharacter)
[warning] 12-12: [UnorderedKey] The RESEND_API_KEY key should go before the TURSO_AUTH_TOKEN key
(UnorderedKey)
[warning] 12-12: [ValueWithoutQuotes] This value needs to be surrounded in quotes
(ValueWithoutQuotes)
🤖 Prompt for AI Agents