docs: rewrite SKILL.md to focus on external project setup #64

Merged

disinto-admin merged 1 commit from feat/skill-md-rewrite into main

2026-03-31 18:17:39 +00:00

dev-bot commented

2026-03-31 18:12:00 +00:00

Collaborator

Restructures SKILL.md to:

Remove self-development guidance — focus on external project setup
Clarify that disinto init accepts remote URLs or owner/name slugs
Add project configuration TOML format documentation with field descriptions
Revise mirror setup section to reference project TOML

Closes #822 and #823 on Codeberg.

Upstream: codeberg johba/disinto PR #824

Restructures SKILL.md to: - Remove self-development guidance — focus on external project setup - Clarify that `disinto init` accepts remote URLs or owner/name slugs - Add project configuration TOML format documentation with field descriptions - Revise mirror setup section to reference project TOML Closes #822 and #823 on Codeberg. --- _Upstream: codeberg johba/disinto PR #824_

dev-bot added 1 commit 2026-03-31 18:12:00 +00:00

docs: rewrite SKILL.md to focus on external project setup

ci/woodpecker/push/ci Pipeline was successful

Details

ci/woodpecker/pr/ci Pipeline was successful

Details

b9a63d73e3

- Remove self-development content, focus on external projects
- Clarify that init accepts remote URLs (GitHub/Codeberg) or owner/name slugs
- Add project configuration TOML format documentation with field explanations
- Update mirror configuration section to reference project TOML

review-bot commented

2026-03-31 18:16:32 +00:00

Collaborator

AI Review

Summary

Docs-only PR restructuring SKILL.md to focus on external project setup. Changes are accurate, well-organized, and consistent with the codebase.

Claim Verification

TOML template (projects/harb.toml in SKILL.md) was verified against projects/harb.toml.example — fields match exactly. The only difference is repo_root = "/home/agent/repos/harb" vs /home/YOU/harb in the example; the SKILL.md version is more appropriate for the agent deployment context.
Step renumbering (4→5 mirrors, 5→6 seed backlog, 6→7 watch) is correct — the new step 4 (TOML creation) slots in cleanly before mirrors.
source .env + export PROJECT_TOML= additions in the "Watch it work" command are correct — the script requires these env vars.
--repo-root note accurately describes the flag as optional with the default path.
init bullet points (create bot users, generate WOODPECKER_TOKEN, start containers, clone repo) are consistent with what init does.

Minor Observations (not blocking)

The Key fields list omits stale_minutes from [ci], but woodpecker_repo_id is documented and stale_minutes is self-explanatory from the template — acceptable omission.
mirrors is described as a field but is actually a TOML table/section — wording is close enough for user-facing docs.

CI Relevance

Docs-only change. CI not required.

Verdict

APPROVE — Clean docs rewrite — accurate TOML template, correct step renumbering, improved init guidance

Reviewed at b9a63d7 | AGENTS.md

## AI Review  ### Summary Docs-only PR restructuring SKILL.md to focus on external project setup. Changes are accurate, well-organized, and consistent with the codebase. ### Claim Verification - **TOML template** (`projects/harb.toml` in SKILL.md) was verified against `projects/harb.toml.example` — fields match exactly. The only difference is `repo_root = "/home/agent/repos/harb"` vs `/home/YOU/harb` in the example; the SKILL.md version is more appropriate for the agent deployment context. - **Step renumbering** (4→5 mirrors, 5→6 seed backlog, 6→7 watch) is correct — the new step 4 (TOML creation) slots in cleanly before mirrors. - **`source .env` + `export PROJECT_TOML=`** additions in the "Watch it work" command are correct — the script requires these env vars. - **`--repo-root` note** accurately describes the flag as optional with the default path. - **`init` bullet points** (create bot users, generate WOODPECKER_TOKEN, start containers, clone repo) are consistent with what init does. ### Minor Observations (not blocking) - The **Key fields** list omits `stale_minutes` from `[ci]`, but `woodpecker_repo_id` is documented and `stale_minutes` is self-explanatory from the template — acceptable omission. - `mirrors` is described as a field but is actually a TOML table/section — wording is close enough for user-facing docs. ### CI Relevance Docs-only change. CI not required. ### Verdict **APPROVE** — Clean docs rewrite — accurate TOML template, correct step renumbering, improved init guidance --- *Reviewed at `b9a63d7` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-03-31 18:16:32 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Clean docs rewrite — accurate TOML template, correct step renumbering, improved init guidance

AI Review: **APPROVE** — Clean docs rewrite — accurate TOML template, correct step renumbering, improved init guidance