chore: move skills to .agents, add release-cog, and enable prettier

[markphelps]

Summary

• move skill definitions to .agents/skills and update related skill locations
• add the new release-cog skill for release workflow guidance
• add npm:prettier to mise and run Prettier in fmt:docs/fmt:docs:fix
• apply formatter fixes across Markdown docs in a follow-up commit

[ask-bonk]

This PR adds Prettier to the docs formatting workflow and reformats all Markdown files.

Summary: Add Prettier 3.6.2 to mise tooling and integrate it into fmt:docs/fmt:docs:fix tasks with proper symlink handling.

Issues found:

1. High: The array emptiness check uses a non-idiomatic pattern. ${files[*]-} only applies the default when unset, but files=() initializes it, so the - never triggers. While it accidentally works, it's fragile.

2. Medium: When no Markdown files exist, prettier runs with no arguments and errors. The check should use ${#files[@]} instead.

[ask-bonk]

Use idiomatic array length check here as well:

Suggested change

	'''
	if [ ${#files[@]} -gt 0 ]; then
	prettier --write "${files[@]}"
	fi

[ask-bonk]

Posted review comments to PR #2961 with two suggestions to fix the array emptiness check in mise.toml.

github run

[ask-bonk]

The latest commit changed the array check pattern, but it's still non-idiomatic. Using ${#files[@]} is the standard bash idiom for checking array emptiness and is more readable.

[ask-bonk]

Use idiomatic array length check for clarity:

Suggested change

	if [ "${files[0]+x}" ]; then
	if [ ${#files[@]} -gt 0 ]; then
	prettier --check "${files[@]}"
	fi

[ask-bonk]

Use idiomatic array length check here as well:

Suggested change

	if [ "${files[0]+x}" ]; then
	if [ ${#files[@]} -gt 0 ]; then
	prettier --write "${files[@]}"
	fi

[ask-bonk]

Posted review with suggestions to fix the array emptiness checks in mise.toml lines 401 and 420. The current "${files[0]+x}" pattern works but ${#files[@]} is the standard bash idiom.

github run

[ask-bonk]

This PR organizes skills into .agents/skills/, adds a new release-cog skill with comprehensive release documentation, and integrates Prettier into the docs formatting workflow.

One issue to address:

The array emptiness check in mise.toml (lines 401 and 420) uses a non-idiomatic bash pattern. Using ${#files[@]} is the standard, more readable idiom for checking if an array has elements.

[ask-bonk]

Use idiomatic array length check for clarity and consistency:

Suggested change

	if [ "${files[0]+x}" ]; then
	if [ ${#files[@]} -gt 0 ]; then
	prettier --check "${files[@]}"
	fi

[ask-bonk]

Use idiomatic array length check here as well:

Suggested change

	if [ "${files[0]+x}" ]; then
	if [ ${#files[@]} -gt 0 ]; then
	prettier --write "${files[@]}"
	fi

[ask-bonk]

I'm Bonk, and I've done a quick review of your PR.

This PR moves skill definitions to .agents/skills/, adds a new release-cog skill with comprehensive release documentation, and integrates Prettier into the docs formatting workflow.

One issue found:

1. Medium severity: The array emptiness checks in mise.toml (lines 401 and 420) use a non-idiomatic bash pattern [ "${files[0]+x}" ]. The standard idiom is [ ${#files[@]} -gt 0 ], which is more readable and clearer in intent.

I've posted suggestions to fix both occurrences.

github run

[ask-bonk]

LGTM

github run