Skip to content

GH#12740: tighten cloudron-app-publishing-skill.md to 150 lines#8

Open
johnwaldo wants to merge 1 commit intomainfrom
simplification/GH-12740-cloudron-publishing
Open

GH#12740: tighten cloudron-app-publishing-skill.md to 150 lines#8
johnwaldo wants to merge 1 commit intomainfrom
simplification/GH-12740-cloudron-publishing

Conversation

@johnwaldo
Copy link
Copy Markdown
Owner

Summary

  • Reduces .agents/tools/deployment/cloudron-app-publishing-skill.md from 178 → 150 lines (28 lines removed, -16%)
  • Removes duplicate Prerequisites section (content already in Quick Reference)
  • Removes Typical Release Cycle section (duplicated the Workflow section at top)
  • Tightens Initialize intro prose (removes redundant sentence)
  • Removes trailing redundant note on manifest fields table
  • All commands, URLs, code blocks, and unique agent content preserved

Verification

  • Line count: wc -l = 150 ✓
  • All 8 URLs present ✓
  • All cloudron commands preserved ✓
  • All 7 code blocks preserved ✓
  • No task IDs or incident references existed in original to preserve

Runtime Testing

  • Testing level: self-assessed
  • Risk classification: low (docs/agent prompt only, no code)
  • Behaviour unchanged: agent instructions identical in substance

Closes marcusquinn#12740

… lines (GH#12740)

Remove duplicate Prerequisites section (already in Quick Reference), collapse
Typical Release Cycle (duplicated Workflow), tighten Initialize intro prose,
and remove redundant trailing note on manifest fields table. All commands,
URLs, code blocks, and unique agent content preserved.
@github-actions
Copy link
Copy Markdown

Maintainer Gate: BLOCKED

This PR cannot be merged because:

Issue marcusquinn#12740: API call failed — cannot verify assignee/labels.
What to do:

  1. A maintainer must review the linked issue(s) and remove the needs-maintainer-review label
  2. The issue(s) must have an assignee before work begins
  3. This check re-runs automatically when the issue is updated

This is an automated check. See issue-triage-gate.yml for the triage policy.

@github-actions
Copy link
Copy Markdown

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 603 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Sun Mar 29 08:21:45 UTC 2026: Code review monitoring started
Sun Mar 29 08:21:46 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 603

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 603
  • VULNERABILITIES: 0

Generated on: Sun Mar 29 08:21:48 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

johnwaldo pushed a commit that referenced this pull request Mar 29, 2026
…usquinn#13373)

* fix: restore traceability and signature rules to build.txt

The signature footer instruction was moved from build.txt to AGENTS.md in
382ed63, after which the model started composing signatures inline —
getting the runtime name wrong ('Claude Code' instead of 'OpenCode') and
omitting version numbers, tokens, and session time.

Root cause: build.txt is the highest-priority instruction set. Rules moved
to AGENTS.md get lost in context during long sessions. The model then
fabricates the signature from its own context instead of calling the helper.

Changes:
- Restore traceability constraints (PR title, Closes #NNN) to build.txt
- Add signature footer as Error Prevention #8 with explicit NEVER-compose rule
- AGENTS.md retains procedural details, points to build.txt for hard rules

* fix: use ref:GH#NNN (colon) not ref=GH#NNN (equals) in build.txt

Aligns with AGENTS.md, TODO.md (884 colon vs 13 equals), and
issue-sync-helper.sh parser which expects colon syntax.
Pre-existing inconsistency caught by CodeRabbit.
johnwaldo pushed a commit that referenced this pull request Mar 30, 2026
…arcusquinn#12451)

- Shorten verbose title
- Remove standalone 80/20 Rule section; merge into principle #8
- Remove filler intro sentence in Document Map
- Collapse 6 '## Quick Reference: X' headers into single '## Quick Reference' with subsections
- Compress intro prose in Core Principles
- All code blocks, specs, benchmarks, links, and chapter refs preserved
johnwaldo pushed a commit that referenced this pull request Mar 30, 2026
…usquinn#13373)

* fix: restore traceability and signature rules to build.txt

The signature footer instruction was moved from build.txt to AGENTS.md in
382ed63, after which the model started composing signatures inline —
getting the runtime name wrong ('Claude Code' instead of 'OpenCode') and
omitting version numbers, tokens, and session time.

Root cause: build.txt is the highest-priority instruction set. Rules moved
to AGENTS.md get lost in context during long sessions. The model then
fabricates the signature from its own context instead of calling the helper.

Changes:
- Restore traceability constraints (PR title, Closes #NNN) to build.txt
- Add signature footer as Error Prevention #8 with explicit NEVER-compose rule
- AGENTS.md retains procedural details, points to build.txt for hard rules

* fix: use ref:GH#NNN (colon) not ref=GH#NNN (equals) in build.txt

Aligns with AGENTS.md, TODO.md (884 colon vs 13 equals), and
issue-sync-helper.sh parser which expects colon syntax.
Pre-existing inconsistency caught by CodeRabbit.
johnwaldo pushed a commit that referenced this pull request Mar 30, 2026
…arcusquinn#12451)

- Shorten verbose title
- Remove standalone 80/20 Rule section; merge into principle #8
- Remove filler intro sentence in Document Map
- Collapse 6 '## Quick Reference: X' headers into single '## Quick Reference' with subsections
- Compress intro prose in Core Principles
- All code blocks, specs, benchmarks, links, and chapter refs preserved
johnwaldo pushed a commit that referenced this pull request Apr 3, 2026
…briefs

Update epic and subtask briefs with:
- Three concurrency modes: sequential, population-based, multi-dimension
- Mailbox integration via mail-helper.sh discovery messages + convoy grouping
- Inter-agent communication for concurrent multi-dimension research
- Updated estimates (~25h total, up from ~20h)
- Cross-dimension campaign summaries
- Concurrency question (#8) in interactive setup flow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

simplification: tighten agent doc Cloudron App Publishing (.agents/tools/deployment/cloudron-app-publishing-skill.md, 178 lines)

1 participant