Port core workflow skills into davidsulitzer.com#6
Conversation
Port 11 selected skills into .cursor/.codex and retarget references so planning, QA, issue, and release workflows are available in this repository too.
![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: 06c1269a51
ℹ️ 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".
| 2. Collect inputs up front: prompt(s), exact text (verbatim), constraints/avoid list, and any input image(s)/mask(s). For multi-image edits, label each input by index and role; for edits, list invariants explicitly. | ||
| 3. If batch: write a temporary JSONL under tmp/ (one job per line), run once, then delete the JSONL. | ||
| 4. Augment prompt into a short labeled spec (structure + constraints) without inventing new creative requirements. | ||
| 5. Run the bundled CLI (`scripts/image_gen.py`) with sensible defaults (see references/cli.md). |
There was a problem hiding this comment.
This skill requires a bundled CLI and companion docs (scripts/image_gen.py and multiple references/*.md files), but this commit only adds SKILL.md; repo-wide search (rg --files -uu) shows none of those referenced paths exist. When generate-assets is invoked, the workflow immediately blocks at its primary execution step, so the skill cannot perform the task it routes to.
Useful? React with 👍 / 👎.
| - Save the returned `slot` value (example: `logTest4`) as `<LOGTEST_SLOT>` | ||
| - If claim fails with "no slots available", wait and retry; do not deploy to a random slot | ||
|
|
||
| 2. Modify `api/api/general.js` (or `davidsulitzer.com/api/api/general.js` from davidsulitzer.com root): |
There was a problem hiding this comment.
Phase 4 tells agents to edit api/api/general.js, but this repository has no api/ directory at all (rg --files -uu contains no such file). In practice, anyone following this skill will fail at this step with a missing-file error, so the documented QA/deploy workflow is not executable in this repo.
Useful? React with 👍 / 👎.
Replace Mixpanel references with provider-agnostic analytics wording and make release analytics steps optional so the skill set fits davidsulitzer.com without Mixpanel dependencies.
|
Follow-up update: removed Mixpanel-specific assumptions from the migrated skills so they better match this personal project setup. Detailed changes in this push:
If you want, I can do one more polish pass to make the analytics sections even leaner (e.g., add a “skip entirely if no analytics backend is configured” branch in each affected skill). |
Remove all migrated Cursor skill copies under .cursor/skills while preserving the .codex skill set as the single source of truth.
|
Follow-up update per request: removed all Cursor skill copies and kept only Codex skills. Detailed changes in this push:
|
1 participant
Summary
.cursor/skillsand.codex/skillsfor davidsulitzer.com.Test plan
.cursor/skills..codex/skills.8Gaston8/davidsulitzer.comreferences.Made with Cursor