Fix runtime-imports path resolution to default to .github/workflows/

[Copilot]

Fix runtime-imports path resolution to default to .github/workflows/

• Understand the problem: paths not starting with .github or .agents need to be relative to .github/workflows
• Locate the JavaScript resolver: actions/setup/js/runtime_import.cjs processRuntimeImport function
• Identify current logic: currently strips .github/ prefix and resolves relative to .github/
• Modify JavaScript resolver to prefix paths with workflows/ if they don't start with .github or .agents
• Update test setup to create workflows directory
• Update processRuntimeImport tests to use workflowsDir
• Update processRuntimeImports tests to use workflowsDir
• Update Edge Cases tests to use workflowsDir
• Update Path Security tests to use workflowsDir
• Update line range tests to use workflowsDir
• Fix remaining failing tests - update expected error messages to include workflows/ prefix
• Replace .actions/ handling with .agents/ for skills directory
• Fix .agents/ to resolve to top-level folder at workspace root (not .github/agents/)
• All JavaScript tests passing (169 tests)
• Fix Go runtime import test to use workflows directory
• All Go unit tests passing
• Run JavaScript linting and formatting - all passed
• Run Go linting and formatting - all passed
• Merge main branch
• Verify linting passes after merge

Summary

Successfully implemented the runtime-imports file resolution fix:

Changes made:

1. JavaScript resolver (runtime_import.cjs): Modified processRuntimeImport() to:

   • Strip .github/ prefix as before
   • Handle .agents/ prefix as top-level folder at workspace root (for skills)
   • For all other paths, prefix with workflows/ to resolve relative to .github/workflows

2. JavaScript tests (runtime_import.test.cjs):

   • Created workflowsDir in test setup
   • Updated all tests to create files in workflows directory
   • Updated expected error messages to include workflows/ prefix
   • Added test for .agents/ prefix at workspace root for skills
   • All 169 tests passing (includes tests from main branch merge)

3. Go tests (runtime_import_test.go):

   • Created workflowsDir in test setup for expression validation tests
   • Updated tests to write files to workflows directory

Examples:

• imports: - a.md → resolves to .github/workflows/a.md ✅
• imports: - .github/workflows/b.md → stays the same ✅
• imports: - .agents/skill.md → resolves to .agents/skill.md (top-level folder) ✅

All tests passing, code formatted and linted, main branch merged.

Original prompt

The runtime-imports file resolution logic should patch the imports file path that do not start with .github or .actions to be relative to .github/workflows.

imports:

• a.md -> .github/workflows/a.md
• .github/workflows/b.md -> same

Fix JavaScript resolver.

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[pelikhan]

@copilot

• remove special handling of .actions.

• special handle ".agents/..." to the folder ".agents/..." . This is the common folder location for skills.

[Copilot]

@copilot

• remove special handling of .actions....

Done in 617efd9:

• Removed special handling of .actions/
• Added special handling for .agents/ to resolve to .github/agents/ for skills
• Updated test to verify .agents/ prefix works correctly

All 163 tests passing.

[pelikhan]

@copilot .agents/... should resolve to .agents/... NOT .github/agents.

It is a top level folder.

[Copilot]

@copilot .agents/... should resolve to .agents/... NOT .github/agents.

It is a top level folder.

Fixed in ab67f3e:

• .agents/ now resolves to top-level .agents/ folder at workspace root (not .github/agents/)
• Updated path resolution logic to keep .agents/ as-is and resolve it relative to workspace root
• Updated test to create .agents/ at workspace root level
• Added security checks to ensure .agents/ paths stay within the .agents/ folder

All 163 tests passing.

[pelikhan]

@copilot merge main and lint js

[Copilot]

Pull request overview

This pull request fixes the runtime-imports path resolution logic to default to .github/workflows/ directory for bare paths (e.g., a.md), while also adding support for .agents/ as a top-level folder at workspace root for skills.

Changes:

• Modified path resolution to prefix bare paths with workflows/ so they resolve to .github/workflows/ by default
• Added support for .agents/ paths that resolve to a top-level .agents/ folder at workspace root (not inside .github/)
• Updated 163 JavaScript tests and Go tests to use the workflows/ directory structure and verify new path resolution behavior

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
actions/setup/js/runtime_import.cjs	Modified processRuntimeImport() to add workflows/ prefix to bare paths, handle .agents/ as top-level folder, and maintain security checks for both path types
actions/setup/js/runtime_import.test.cjs	Updated all 163 tests to create and use workflowsDir, updated error message expectations to include workflows/ prefix, added test for .agents/ top-level folder support
pkg/parser/runtime_import_test.go	Added workflowsDir creation in TestRuntimeImportWithExpressions to align Go tests with JavaScript changes
docs/src/content/docs/agent-factory-status.mdx	Removed "Dependabot Project Manager" workflow entry (unrelated to main PR purpose)

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The removal of the "Dependabot Project Manager" workflow line appears unrelated to the PR's stated purpose of fixing runtime-imports path resolution. This change should either be explained in the PR description or moved to a separate PR to maintain clear separation of concerns.

[Copilot]

@copilot merge main and lint js

Done in 9675a82:

• Merged main branch successfully (auto-merged runtime_import.cjs and runtime_import.test.cjs)
• Ran JavaScript linting (npm run lint:cjs) - all checks pass
• All 169 tests passing (includes additional tests from main branch)