Works post-merge.

And yeah, I love the brew terminology. Promoting alcoholism since 2009.

Please provide an example file that reproduces the failure. This AI generated explanation is mostly a hallucination and a waste of time.

i.e. selectAll+Delete+UndoDelete

Just so the passing reader is clear, I'm aware that this PR is moderately insane. Amusingly it does pass all of our tests though

Thanks for adding this documentation. I had been ignoring all the DNS failure warnings for ages!

Reached max QA-fix rounds (2). Escalating to human for review.

On further thought: not demoting to a warning, since this is clear UB. Wontfix, then. 'tis a feature ;)

🚫 Duplicated proposal withdrawn by 🤖 ProposalPolice.

lgtm (it's a manual job so it won't break anything)

no need for locks? It looks like we do it in the right order, but I'm not sure how much c++ is allowed to screw us if we don't take locks.

Fixed in 26d317be: cd /work now fails fast with a FATAL message.

Is it reasonable to include in this pr how a customer would use the skill? Is it something like.. /plugin marketplace add anthropics/skills, select oadp-cli etc.. and BOOOM .. you gots it?