Repository navigation
Conversation
Règles de périmètre figé, ASO/SEO, main sans nouveau worktree, contrat .sceo/contract.json, verrou Xcode et relecture avant qualification finale. Co-authored-by: Cursor <cursoragent@cursor.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 740435c61b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - **Périmètre figé** au démarrage : P0/P1 prévus ; P2 → cartes. Seul un nouveau P0 crash/paiement entre (requalification ciblée). | ||
| - **ASO/SEO** : examinés à chaque chantier ; bloquants pour la soumission seulement si fiche App Store ou offre changent. Mots-clés 95–100 caractères : objectif non bloquant. | ||
| - **Git** : checkout principal sur `main`, un seul écrivain, aucun nouveau worktree (préserver les actifs jusqu'à clôture). Une branche et une PR par changement (`gh`). PR uniquement documentaire : merge autonome après une relecture croisée par un autre moteur. | ||
| - **Contrat** : à l'admission, `sceo-contract-check` valide `.sceo/contract.json`. iPhone-only : aucun format iPad. Au plus 5 captures par langue et par format requis. |
There was a problem hiding this comment.
For any chantier started from a fresh checkout, this mandatory admission step has no input to validate: a repo-wide tree search finds neither .sceo/contract.json nor a generator for it. Consequently the newly documented process cannot pass its first gate; add the contract to the repository or make this rule conditional on a contract supplied by the external studio environment.
Useful? React with 👍 / 👎.
| - **ASO/SEO** : examinés à chaque chantier ; bloquants pour la soumission seulement si fiche App Store ou offre changent. Mots-clés 95–100 caractères : objectif non bloquant. | ||
| - **Git** : checkout principal sur `main`, un seul écrivain, aucun nouveau worktree (préserver les actifs jusqu'à clôture). Une branche et une PR par changement (`gh`). PR uniquement documentaire : merge autonome après une relecture croisée par un autre moteur. | ||
| - **Contrat** : à l'admission, `sceo-contract-check` valide `.sceo/contract.json`. iPhone-only : aucun format iPad. Au plus 5 captures par langue et par format requis. | ||
| - **Mac** : 2 jobs `sceo-run` max ; une opération Xcode lourde à la fois (`sceo-xcode-lock`). Compiler les tests après chaque modif native ; terminé seulement si `build-for-testing` et tests verts sur `$SCEO_SIM_UDID`. |
There was a problem hiding this comment.
When native RealmBox code such as apps/desktop/src-tauri changes, this rule makes completion depend on build-for-testing and $SCEO_SIM_UDID, but the reviewed tree contains no Xcode project/workspace or iOS target; the checked macOS workflow instead builds the Tauri desktop application with pnpm and Cargo. Such changes can therefore never satisfy the new completion criterion, so use the repository's native test/build commands here or explicitly limit this gate to projects that actually provide an Xcode simulator target.
Useful? React with 👍 / 👎.
|
Relecture croisée (Grok Bot, autre moteur que le job Cursor composer-2.5) : diff 1 fichier, AGENTS.md uniquement, règles du plan commun du 10/10 conformes. OK pour merge. |
This branch has not been deployed
1 participant
Summary
Test plan
Made with Cursor