al-pr-prepare
Skill115 starsChanged 12 days agoMITView on GitHub
Prepare a clean, documented pull request draft for AL features or fixes with summary, testing notes, and checklist. ALDC workflow (Copilot prompt al-pr-prepare); invoke explicitly.
In this repository55 other files it gives its agents.
- CLAUDE.mdCLAUDE.md
- claude-plugin/CLAUDE.mdCLAUDE.md
- .github/copilot-instructions.mdCopilot instructions
- instructions/copilot-instructions.mdCopilot instructions
- agent-builderSkill
- al-agent-build-instructionsSkill
- al-agent-createSkill
- al-agent-instructionsSkill
- al-agent-taskSkill
- al-agent-testSkill
- al-buildSkill
- al-context-createSkill
- al-initializeSkill
- al-memory-createSkill
- al-spec-createSkill
- architectSkill
- auditSkill
- conductSkill
- developSkill
- presalesSkill
- reviewSkill
- skill-agent-instructionsSkill
- skill-agent-task-patternsSkill
- skill-agent-toolkitSkill
- skill-al-review-pipelineSkill
- skill-apiSkill
- skill-contribution-assistantSkill
- skill-copilotSkill
- skill-debugSkill
- skill-estimationSkill
- skill-eventsSkill
- skill-migrateSkill
- skill-pagesSkill
- skill-performanceSkill
- skill-permissionsSkill
- skill-testingSkill
- skill-translateSkill
- specSkill
- triageSkill
- skill-agent-instructionsSkill
- skill-agent-task-patternsSkill
- skill-agent-toolkitSkill
- skill-al-review-pipelineSkill
- skill-apiSkill
- skill-contribution-assistantSkill
- skill-copilotSkill
- skill-debugSkill
- skill-estimationSkill
- skill-eventsSkill
- skill-migrateSkill
- skill-pagesSkill
- skill-performanceSkill
- skill-permissionsSkill
- skill-testingSkill
- skill-translateSkill
The file
We split this skill into the rules it gives an agent
8 rules. About 0 are common, 4 are distinctive, and 4 are very specific to this project.
≈0
Common · estimate
Many files like this have a rule of this kind.
≈4
Distinctive · estimate
This author's own choices. Worth a look.
4
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Distinctive: line 48
Quality validation: - Verify tests pass and build succeeds
- Very specific: line 57
Use codebase to analyze modifications:
- Very specific: line 62
Use githubRepo to gather context:
- Distinctive: line 71
Commit messages and references
- Distinctive: line 80
Tests: - Test codeunits, scenarios, data
- Very specific: line 98
If <Reviewer from $ARGUMENTS> is specified, include in the draft.
- Very specific: line 102
Create /reports/pr-draft.md with this structure:
- Distinctive: line 348
Use tables for structure
≈ “Common” is an estimate: at least 1 in 5 of the 350,495 skill files we read have a rule of the same kind, not the same words. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
Claude Code adapter — generated by
scripts/sync-plugin-support.js, do not edit. Canonical source:prompts/al-pr-prepare.prompt.md(sha256e6c5e60c501f779c8e3d899bc7d3f326a3fe0d545c767540a48f07ebc483e8ab). This workflow was written for@workspace use …in GitHub Copilot Chat. Apply these harness mappings and treat everything else as binding:
Copilot surface in the contract Claude Code equivalent #runSubagent/runSubagent/agents:The Tasktool withsubagent_type: "aldc:<agent>"; one invocation carries everything inline (subagents are stateless)#askQuestions/vscode/askQuestionsAskUserQuestion(main session only; a subagent returns its questions in its structured output)handoffs:entries, andsend: falseon oneNo equivalent — the gate is yours to keep. In Copilot a handoff is a button the human clicks, and send: falseadditionally hands them the prompt to review before it is sent; the host supplies the approval. Claude Code delegates through theTasktool, with no click and no review step. So never auto-delegate: present your output, get explicit approval, and only then delegate or route to another role. The handoff's routing is carried by the plugin's role entry skills; its human gate is carried by this rule@al-architect,@al-spec-agent,@al-conductor,@al-developer,@al-developer-reviewer,@al-triage,@al-presales,@al-agent-builder,@dredd/aldc:architect,/aldc:spec,/aldc:conduct,/aldc:develop,/aldc:review,/aldc:triage,/aldc:presales,/aldc:agent-builder,/aldc:audit(orclaude --agent aldc:<agent>)@workspace use <workflow>/aldc:<workflow>with dots replaced by dashes (al-spec.create->/aldc:al-spec-create); already rewritten below${input:Name}prompt variablesTake the value from $ARGUMENTS; ask throughAskUserQuestionwhen missing. Already rewritten below as<Name from $ARGUMENTS>execute/runInTerminal, AL compilation or symbol restoreDeveloper/Implementer: discovered official AL MCP mcp__al__al_compile/mcp__al__al_build/mcp__al__al_downloadsymbols, or an installed ALTool/AL-Go runner throughBash. Use the correct App/Test project; unavailable capabilities are never simulated. Publishing retains its separate human/CI gate and is not granted by this adapteredit,read/readFile,search,#codebaseEdit/Write,Read,Grep/Glob#changes,changes,search/changesRead-only git status/git diffthroughBashonly where granted; otherwise read the caller-supplied diff artifact or return the missing evidence requestread/problems,al_get_diagnostics,#testFailureDiscovered mcp__al__al_getdiagnosticsfor the explicit project, or actual compiler/test output. Diagnostics are not evidence of a fresh compile or executed tests. No result: recordnot-run/unavailableal-symbols-mcp/*,upstash/context7/*,microsoft-learn/*,microsoft-docs/*Tools named mcp__al-symbols-mcp__*,mcp__context7__*,mcp__microsoft-docs__*(or themcp__plugin_aldc_…form). A server that is not loaded isunavailable; do not simulate itsshadowsdk.al-lsp-for-agents/*,#usages, semantic definition/reference queriesClaude LSPwith the existing AL LSP for Agents companion; discover the actual operations and project context first. VS Code tool identifiers are not Claude tool namesms-dynamics-smb.al/*,vscode.mermaid-chat-features/*VS Code tool identifiers do not exist here. Use only a separately discovered Claude capability; never rename community tools as Microsoft tools vscode/memory,vscode/*Not available; keep state in the canonical .claude/plans/artifacts of the projecttodoTodoWritefor in-session tracking; durable state stays in.claude/plans/"load skill-x"The Skilltool (aldc:<skill-name>) or read${CLAUDE_PROJECT_DIR}/.claude/skills/<skill-name>/SKILL.md.github/instructions/*.instructions.md(applyTo)${CLAUDE_PROJECT_DIR}/.claude/rules/*.md(paths:);/aldc:al-initializecopies them to the project's.claude/rules/for auto-apply in the main session; inject inline for subagents exactly as the contract says.github/skills/,.github/plans/…,docs/templates/${CLAUDE_PROJECT_DIR}/.claude/skills/,.claude/plans/…(project-owned,aldc.yaml -> plans.root),${CLAUDE_PROJECT_DIR}/claude-plugin/docs/templates/; already rewritten belowpython …runtime helpers,--aldc-rootPaths already point at ${CLAUDE_PLUGIN_ROOT}; usepythonorpython3, whichever exists (stdlib only)aldc.yaml${CLAUDE_PROJECT_DIR}/aldc.yamlwhen the project has one, otherwise${CLAUDE_PROJECT_DIR}/claude-plugin/aldc.yamlRead
${CLAUDE_PROJECT_DIR}/claude-plugin/docs/claude-al-tooling.mdfor this surface’s LSP/MCP bindings, then${CLAUDE_PROJECT_DIR}/.claude/skills/skill-migrate/references/cli-al-tools.mdfor the shared role/evidence contract. The Claude guide supersedes older capability-availability examples, never role responsibilities or human gates.
AL Pull Request Preparation
Your goal is to prepare a pull request draft for the branch <Branch from $ARGUMENTS> summarizing all modifications, test evidence, and validation steps.
🔒 Human Gate: Pre-PR Review
Before generating PR draft document:
- Review code changes - Present summary of all modifications
- Security check - Confirm no sensitive data in commits
- Quality validation - Verify tests pass and build succeeds
- Human approval required - Obtain confirmation before creating PR draft
Process
1. Change Analysis
Inspect Branch Differences
Use codebase to analyze modifications:
codebase: Compare <Branch from $ARGUMENTS> with main branchUse githubRepo to gather context:
githubRepo: Get branch information and commit historyGather:
- Modified files and line counts
- New files added
- Deleted files
- Commit messages and references
- Related issues or work items
Classify Changes
Categorize into:
- New Features - New AL objects, functionality, APIs
- Bug Fixes - Corrections, refactors, optimizations
- Tests - Test codeunits, scenarios, data
- Configuration - app.json, permissions, dependencies
- Documentation - README, comments, API docs
2. Extract Metadata
Find References: Scan commit messages for:
- Issue references (#123)
- Work item IDs
- Requirement numbers
Pattern matching:
- Fixes #123
- Closes #456
- Related to WORK-789
Identify Reviewers:
If <Reviewer from $ARGUMENTS> is specified, include in the draft.
3. Generate PR Draft
Create /reports/pr-draft.md with this structure:
# Pull Request: [Feature/Fix Title]
**Branch:** `<Branch from $ARGUMENTS>`
**Target:** `main`
**Author:** [Author Name]
**Date:** [Current Date]
## Overview
[2-3 sentence description of changes]
**Type of Change:**
- [ ] New Feature
- [ ] Bug Fix
- [ ] Refactoring
- [ ] Performance Improvement
- [ ] Documentation
- [ ] Configuration Change
## Related Issues
- Closes #[issue-number]
- Relates to #[issue-number]
## Changes Summary
### File Changes
| Category | Files | +Lines | -Lines |
|----------|-------|--------|--------|
| New Features | [#] | [#] | [#] |
| Bug Fixes | [#] | [#] | [#] |
| Tests | [#] | [#] | [#] |
| Docs | [#] | [#] | [#] |
| **Total** | **[#]** | **[#]** | **[#]** |
### AL Objects Modified
#### New Objects
| Type | ID | Name | Purpose |
|------|----|----- |---------|
| Table | [ID] | [Name] | [Purpose] |
| Page | [ID] | [Name] | [Purpose] |
| Codeunit | [ID] | [Name] | [Purpose] |
#### Modified Objects
| Type | ID | Name | Changes |
|------|----|----- |---------|
| TableExt | [ID] | [Name] | [Description] |
| PageExt | [ID] | [Name] | [Description] |
#### Deleted Objects
| Type | ID | Name | Reason |
|------|----|----- |-------|
| [Type] | [ID] | [Name] | [Reason] |
## Technical Details
### Architecture Changes
[Describe design pattern or architecture changes]
### Dependencies
**New:** [List new dependencies]
**Modified:** [old version → new version]
**Removed:** [List removed dependencies]
### Database Changes
- [ ] New tables
- [ ] New fields
- [ ] Modified fields
- [ ] New keys
**Migration Notes:** [Any data migration needed]
### API Changes
**New Endpoints:**
- `GET /api/[endpoint]` - [Description]
**Modified Endpoints:**
- `[Method] /api/[endpoint]` - [Changes]
**Breaking Changes:** [List breaking changes]
### Events
**Published:** [New events and purpose]
**Subscribed:** [New subscribers and purpose]
## Testing
### Test Scenarios
1. **Scenario:** [Description]
- **Steps:** [How to test]
- **Expected:** [Expected result]
- **Result:** ✅ Pass / ❌ Fail
### Automated Tests
- ✅ Unit Tests: [X/Y passed]
- ✅ Integration Tests: [X/Y passed]
- ✅ Code Coverage: [X]%
### Performance Impact
- [ ] No impact
- [ ] Improvement: [Details]
- [ ] Potential impact: [Mitigation]
## Review Checklist
### Code Quality
- [ ] Follows AL naming conventions
- [ ] Follows AL style guidelines
- [ ] No compiler warnings
- [ ] Error handling implemented
- [ ] Logging adequate
### Security
- [ ] No hardcoded secrets
- [ ] Permissions reviewed
- [ ] Input validation
- [ ] No SQL injection risks
### Testing
- [ ] All tests pass
- [ ] New code has tests
- [ ] Edge cases tested
- [ ] Manual testing done
### Documentation
- [ ] Code comments added
- [ ] API docs updated
- [ ] README updated
- [ ] Help text added
### BC Specific
- [ ] Object IDs in range
- [ ] Event patterns correct
- [ ] Page layouts user-friendly
- [ ] Translations handled
### Deployment
- [ ] Build succeeds
- [ ] Package creates
- [ ] Deployment steps documented
- [ ] Rollback plan ready
## Deployment Notes
### Steps
1. [Deployment instructions]
2. [Configuration changes]
3. [Verification steps]
### Prerequisites
[Requirements for deployment]
### Rollback Plan
[How to rollback if needed]
## Screenshots
### Before
[Previous state]
### After
[New state]
## Additional Notes
### Known Issues
[Limitations or known issues]
### Future Enhancements
[Potential improvements]
### Breaking Changes
[Breaking changes affecting existing functionality]
## Reviewer Notes
**Suggested Reviewers:**
- <Reviewer from $ARGUMENTS> - [Reason]
**Focus Areas:**
1. [Area to review carefully]
2. [Another focus area]
**Questions:**
[Specific questions or concerns]
---
**Generated by:** AL PR Preparation Workflow
**Generated on:** [Timestamp]Output
Primary: /reports/pr-draft.md
Format: Complete markdown document ready for PR creation
Success Criteria
- ✅ PR draft file created under
/reports/pr-draft.md - ✅ Changes summarized by category
- ✅ All modified AL objects documented
- ✅ Related issues referenced
- ✅ Review checklist complete
- ✅ Deployment notes clear
Common PR Patterns
Feature Addition
- Emphasize new capabilities
- User benefit focus
- Comprehensive testing
- Document APIs/events
Bug Fix
- Reference original issue
- Explain root cause
- Show before/after
- Include regression tests
Refactoring
- Explain motivation
- Show no functional changes
- Highlight improvements
- Demonstrate coverage
Performance Optimization
- Include benchmarks
- Show improvements
- Document approach
- Note trade-offs
Tips
- Be concise but thorough
- Use tables for structure
- Include file names and line numbers
- Link to related documentation
- Provide context for changes
- Make reviewer's job easy
- Include visual evidence
- Anticipate questions
- Document decisions
- Keep security visible
Next Steps
For final validation:
@AL Development Conductor # TDD orchestration with review subagent
@AL Implementation Specialist # Direct testing and fixesPR draft ready for GitHub submission.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

