Version bump#78
Merged
Merged
Conversation
Bumps [taiki-e/create-gh-release-action](https://github.com/taiki-e/create-gh-release-action) from 1.9.3 to 1.11.0. - [Release notes](https://github.com/taiki-e/create-gh-release-action/releases) - [Changelog](https://github.com/taiki-e/create-gh-release-action/blob/main/CHANGELOG.md) - [Commits](taiki-e/create-gh-release-action@v1.9.3...v1.11.0) --- updated-dependencies: - dependency-name: taiki-e/create-gh-release-action dependency-version: 1.11.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: Copilot <copilot@github.com>
…/create-gh-release-action-1.11.0 Bump taiki-e/create-gh-release-action from 1.9.3 to 1.11.0
…d manifest section Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/bd6c5d76-044f-4462-b8fb-459c2651fc07 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
… cleaner iteration Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/bd6c5d76-044f-4462-b8fb-459c2651fc07 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
…, string interpolation Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/bd6c5d76-044f-4462-b8fb-459c2651fc07 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
…lder feat: Separate custom skills into `.agents/skills-custom/` with dedicated manifest section
…f .ai Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/34266c38-e042-4cf3-bd14-32c40280e8a6 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/74bcab47-6579-4cb3-baed-75661c6e28e9 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
…checks Fix `coldbox ai doctor` and `coldbox ai uninstall` checking wrong directory (`.ai` vs `.agents`)
…check for updates in the registry, providing better feedback during potentially long-running integrity checks
…in JSON format for easier parsing in scripts and CI pipelines
… installed registry skills with per-skill feedback and progress updates - `coldbox ai skills update <name>` command to re-download and overwrite a single installed registry skill by local name
Add an `excludes` array to the manifest that tracks skill names the user has explicitly removed via `coldbox ai skills remove`. During `refresh()`, skills in the excludes list are filtered out of `missingDesiredSkills` so they are never auto-reinstalled. When a skill is explicitly re-installed via `coldbox ai skills install`, its name is removed from `excludes` so normal auto-management resumes. - SkillManager.refresh(): initialise excludes section; filter excluded skills from missingDesiredSkills (step 0) - SkillManager.removeSkillFromProject(): append removed skill name to manifest.excludes before saving - SkillManager.installRemoteSkill(): lift exclusion when skill is explicitly re-installed - SkillManager.ensureExcludesSection(): new private helper for backwards-compatible initialisation of manifest.excludes Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/92d13e96-d882-4c14-9d67-62579d852353 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
…aram Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/92d13e96-d882-4c14-9d67-62579d852353 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/9646a1c2-ec9e-48db-9ef0-4d0d4d75f87a Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
fix: prevent auto-reinstall of explicitly removed skills on refresh
Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/0aaba08f-24cc-42d5-a5ce-76b946533733 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
…ions Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/d4c4e167-5ab3-4436-a91f-9cd8a0a58a0e Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
Add missing DB_SCHEMA to .env template
Fix `coldbox ai refresh` custom-skill manifest crash and preserve AGENTS.md content outside managed markers
Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/c18c79bb-9a9f-4f05-810e-28c19f365620 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
Agent-Logs-Url: https://github.com/ColdBox/coldbox-cli/sessions/55e138bc-afc1-409f-b814-c03cb604d2f0 Co-authored-by: lmajano <137111+lmajano@users.noreply.github.com>
Propagate `--boxlang` / `--noboxlang` from `create handler` to generated views
…setting Honor server.json webroot server discovery in `coldbox watch-reinit`
Member
Author
|
@copilot resolve the merge conflicts in this pull request |
Contributor
|
@lmajano I can’t apply these changes because the pull request’s branch is the repository’s default branch, which I’m not allowed to push to directly. You can ask me to create a new pull request with the requested changes instead and I'll open a separate PR on a new branch. |
There was a problem hiding this comment.
Pull request overview
This PR expands ColdBox CLI’s AI integration by introducing a separate, project-committable custom skills location and adding skill update capabilities, alongside several scaffolding and template/documentation adjustments.
Changes:
- Split AI skills into core/framework (
.agents/skills/, auto-managed) vs project-authored (.agents/skills-custom/, commit-friendly) and propagate manifest/schema updates. - Add
coldbox ai skills update(single or all skills), plus enhancements to skills listing (JSON output, progress feedback). - Misc updates to scaffolding commands/templates (handler view language propagation, model test path prefixing, watch-reinit server discovery) and documentation/instructions relocation (AGENTS.md / CLAUDE.md).
Reviewed changes
Copilot reviewed 11 out of 12 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| templates/env.example | Adds DB_SCHEMA example env var. |
| templates/ai/guidelines/core/boxlang.md | Updates BoxLang guideline content and examples (annotations, casting, etc.). |
| templates/ai/agents/agent-modern-instructions.md | Updates AI directory/manifest/docs to .agents and introduces skills-custom/. |
| templates/ai/agents/agent-flat-instructions.md | Updates AI directory/manifest/docs to .agents and introduces skills-custom/. |
| skills-lock.json | Adds/updates locked skill metadata entries and hashes. |
| models/SkillManager.cfc | Implements skills-custom/, manifest customSkills[] + excludes[], and updated lookup/removal behavior. |
| models/BaseCommand.cfc | Adds getAppPrefixDot() helper for dot-notation prefixes. |
| models/BaseAICommand.cfc | Adds injected formatterUtil and keeps AI command helpers centralized. |
| models/AIService.cfc | Adds customSkills to manifest/info/stats and mirrors MCP config to .vscode/mcp.json for Copilot. |
| models/AgentRegistry.cfc | Updates managed-section merge behavior to preserve user content before/after markers; updates skills rendering. |
| commands/coldbox/watch-reinit.cfc | Improves server discovery to honor server.json webroot discovery. |
| commands/coldbox/create/model.cfc | Fixes generated model test path to include correct app/models prefix (dot-notation). |
| commands/coldbox/create/handler.cfc | Adds boxlang arg propagation to generated views so extensions match language mode. |
| commands/coldbox/create/app.cfc | Renames “modern” skeleton option to “tiered” and updates Vite eligibility checks/docs. |
| commands/coldbox/create/app-wizard.cfc | Updates wizard option from “modern” to “tiered”. |
| commands/coldbox/ai/uninstall.cfc | Updates uninstall messaging/paths from .ai to .agents. |
| commands/coldbox/ai/skills/update.cfc | New command to re-download/update one or all installed registry skills. |
| commands/coldbox/ai/skills/remove.cfc | Updates docs to reflect removal checks in both skills directories. |
| commands/coldbox/ai/skills/override.cfc | Moves overrides to skills-custom/. |
| commands/coldbox/ai/skills/list.cfc | Adds --json, progress feedback for --outdated, and custom skills grouping. |
| commands/coldbox/ai/skills/install.cfc | Adjusts registry listing/selection behavior and keeps resolved skill metadata richer. |
| commands/coldbox/ai/skills/create.cfc | Creates custom skills under skills-custom/. |
| CLAUDE.md | Adds pointer file directing users to AGENTS.md. |
| changelog.md | Documents fixes/additions and updates compare links. |
| box.json | Bumps module version. |
| AGENTS.md | New consolidated AI coding instructions document (migrated from Copilot instructions). |
| .gitignore | Ignores .agents/** and .claude/** paths. |
| .github/workflows/release.yml | Updates GitHub release action version. |
| .github/copilot-instructions.md | Removes old Copilot instructions file (content moved to AGENTS.md). |
| .agents/skills/testbox-unit-xunit/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-runners/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-reporters/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-mockbox/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-listeners/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-expectations/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-cbmockdata/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-bdd/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/testbox-assertions/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/coldbox-testing-fixtures/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/coldbox-testing-coverage/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/coldbox-docbox-generation/SKILL.md | Adds vendored skill documentation content. |
| .agents/skills/coldbox-docbox-annotations/SKILL.md | Adds vendored skill documentation content. |
Comments suppressed due to low confidence (1)
AGENTS.md:176
- This section appears duplicated ("Template Testing" / "Module Dependencies" / "Key Patterns & Conventions" / "Testing & Build" repeat), and there is a broken sentence ("...modern layout. handlers via..."). Please deduplicate and fix the malformed line to avoid confusing contributors.
## Testing & Build
**Build Scripts** (from `box.json`):
- `box build:module` — Full module build via `/build/Build.cfc`
- `box build:docs` — Generate documentation via DocBox
- `box format` — Run CFFormat on commands, models, build, and ModuleConfig.cfc
- `box format:watch` — Watch and auto-format on file changes
- `box format:check` — Check formatting without modifying files
- `box release` — Run release recipe (`build/release.boxr`)
**Template Testing**: The `/tests/` directory contains sample module structure for testing generated code patterns. The `/testapp/` directory contains a full test application with modern layout. handlers via template selection
- Modern templates (`boxlang`, `modern`) support additional features via flags: `--vite`, `--rest`, `--docker`, `--migrations`
- Default skeleton is now `boxlang` instead of `advanced`
**Module Dependencies**: The module lazy-loads `testbox-cli` and `commandbox-migrations` via utility methods `ensureTestBoxModule()` and `ensureMigrationsModule()` only when needed.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+167
to
+170
| // Read existing content and locate markers | ||
| var existingContent = fileRead( filePath ).trim() | ||
| var startPos = findNoCase( startMarker, existingContent ) | ||
| var endPos = findNoCase( endMarker, existingContent ) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Please include a summary of the changes and which issue(s) is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Please note that all PRs must have tests attached to them
IMPORTANT: Please review the CONTRIBUTING.md file for detailed contributing guidelines.
Issues
All PRs must have an accompanied issue. Please make sure you created it and linked it here.
Type of change
Please delete options that are not relevant.
Checklist