Skip to content

build: bump pbxproj to 67 for Stage 3b + nostrconnect-polish TestFlight#38

Merged
DocNR merged 1 commit into
mainfrom
chore/pbxproj-build-67
May 8, 2026
Merged

build: bump pbxproj to 67 for Stage 3b + nostrconnect-polish TestFlight#38
DocNR merged 1 commit into
mainfrom
chore/pbxproj-build-67

Conversation

@DocNR
Copy link
Copy Markdown
Owner

@DocNR DocNR commented May 8, 2026

Summary

Bumps CURRENT_PROJECT_VERSION 66 → 67 so a single TF build covers both merges:

  • PR #37 — Stage 3b ProfileFetcher extension (refactor)
  • PR #36 — Nostrconnect connect-flow polish (same-device overlay copy + QR dedup + docs)

Changes

  • Clave.xcodeproj/project.pbxproj — 8 line edits

Mechanical change.

Test plan

🤖 Generated with Claude Code

Internal-only build for on-device verification of two merges:

- Stage 3b ProfileFetcher extraction (#37): 287 LOC moved from
  AppState.swift into Clave/AppState+ProfileFetcher.swift; -36.6%
  AppState size since the start of the refactor sprint.
- Nostrconnect connect-flow polish (#36): switch-back overlay copy
  for same-device pairs (paste path), QR rescan dedup, and
  Universal-Link clarification in docs/nip46-compatibility.md.

Bumps CURRENT_PROJECT_VERSION 66 -> 67 across all 4 targets in both
Debug and Release configs.

On-device verification gates (all in one TF cycle):
- Profile flows: account add/switch/delete, PFP correctness across
  switches, AccountDetailView pull-to-refresh, multi-account profile
  fan-out fetch.
- Same-device nostrconnect via paste -> new "Switch back to your
  client app" subtitle should render.
- Cross-device nostrconnect via QR scan -> original "Stay in Clave"
  subtitle preserved.
- QR cancel-and-rescan loop should now break (dedup against
  lastAcceptedScanCode).

Stage 3c (ProxyClient, ~400 LOC) is the next refactor PR.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@DocNR DocNR merged commit 11c7117 into main May 8, 2026
@DocNR DocNR deleted the chore/pbxproj-build-67 branch May 8, 2026 02:50
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