Skip to content

docs(skills): add management-API testing recipe skill - #632

Closed
devin-ai-integration[bot] wants to merge 5 commits into
devfrom
devin/1790262642-update-skills-v2
Closed

devin-ai-integration[bot] wants to merge 5 commits into
devfrom
devin/1790262642-update-skills-v2

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Summary

  • Adds .agents/skills/testing-opencodex-management-api/SKILL.md — test-recipe knowledge for the management API: isolating client homes in test setup and documenting residual macOS write paths.
  • Supersedes Add testing skill for the OpenCodex management API #616, which is stuck in draft: the review gate's re-attestation after a post-ready push requires an edited event from the PR author with sender type "User", which a bot-authored PR can never produce.

Verification

  • Documentation-only change; no code paths affected.

Checklist

  • Scope stays focused and avoids unrelated cleanup.
  • Docs or release notes were updated when needed.
  • Security-sensitive changes were reviewed for secrets, auth, and unsafe defaults.

Link to Devin session: https://app.devin.ai/sessions/18173a51bcf04824a2e66ca1d2e42ccf
Open in Devin Desktop: https://app.devin.ai/desktop/session/18173a51bcf04824a2e66ca1d2e42ccf?variant=devin
Requested by: @luvs01


Devin Review

Review readiness checklist

This PR stays in draft until every box below is ticked. Tick all four boxes once the requirements are met:

  • Required local validation passed; commands, results, and any full-suite exception are documented.

  • I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).

  • I resolved all correct Codex and CodeRabbit findings.

  • My PR is ready for review.

devin-ai-integration Bot and others added 4 commits September 22, 2026 08:09
Co-authored-by: Epinephrine <luvs01@hanmail.net>
OPENCODEX_HOME relocates only opencodex state; startup still syncs client
homes unless clientIntegrations.* are off AND CODEX_HOME/GROK_HOME/
CLAUDE_CONFIG_DIR/OPENCODEX_CLAUDE_DESKTOP_CONFIG_DIR point at scratch.
codexAutoStart:false never gated those writes.

Co-Authored-By: Epinephrine <luvs01@hanmail.net>
Co-Authored-By: Epinephrine <luvs01@hanmail.net>
@devin-ai-integration

Copy link
Copy Markdown
Author

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@coderabbitai

coderabbitai Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: luvs01/opencodex/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 7bf538af-5009-4550-ac18-fafaf7dc10f2

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

✅ Deterministic PR hygiene checks passed.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Sep 24, 2026
@github-actions

github-actions Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

✅ READY

  • all PR quality gates passed; the review readiness checklist is complete.

Review readiness checklist

  • ✅ Required local validation passed; commands, results, and any full-suite exception are documented.
  • ✅ I pushed my PR to a recent dev commit (at most 10 behind; a maintainer may still ask for the exact tip before merge).
  • ✅ I resolved all correct Codex and CodeRabbit findings.
  • ✅ My PR is ready for review.

✅ 4/4 boxes ticked.

This pull request is already Ready for Review.
The review-ready label marks this PR as ready; review automation runs independently.
Maintainers: @lidge-jun @Ingwannu

@github-actions
github-actions Bot marked this pull request as draft September 24, 2026 15:12
devin-ai-integration[bot]

This comment was marked as resolved.

…merge atomicity

Co-Authored-By: Epinephrine <luvs01@hanmail.net>
@devin-ai-integration

Copy link
Copy Markdown
Author

Fixed in 21ab49c3c. Confirmed refreshOwnedRaycastCatalog runs unconditionally at startup and raycastAiDir resolves ~/.config/raycast/ai under the OS home on both macOS and Windows — the caveat now says "macOS AND Windows" and limits the zshrc note to platforms where a zshrc exists.

@devin-ai-integration

Copy link
Copy Markdown
Author

Fixed in 21ab49c3c. Verified enqueueManualLabRun returns {run, trigger} and that claimNextRun returning null (shutdown, pre-claim abort, claim miss) leaves the record queued — the doc now states run is the post-dispatch record, normally terminal but possibly still queued/cancelled.

@devin-ai-integration

Copy link
Copy Markdown
Author

Fixed in 21ab49c3c. Confirmed the PUT handler reads loadLabAutomationConfig before saveLabAutomationConfig — the doc now says publication is one atomic rename while the read-merge is not under the save lock, so concurrent PUTs can lose one update.

@github-actions
github-actions Bot marked this pull request as ready for review September 24, 2026 15:22
@luvs01

luvs01 commented Sep 27, 2026

Copy link
Copy Markdown
Owner

이관됨: lidge-jun#6051

@luvs01

luvs01 commented Sep 27, 2026

Copy link
Copy Markdown
Owner

동일 수정이 상류 저장소에 제출되어 이 포크 PR의 목적은 달성됐습니다.

@luvs01 luvs01 closed this Sep 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation review-ready

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant