fix: include SkillRoute portfolio in package runtime

Author: rockyshi1993

package.json

@@ -176,6 +176,7 @@
     "instructions.md",
     "plugin.json",
     ".mcp.json",
+    "skills/portfolio.json",
     "agents/",
     "skills/*/**",
     "instructions/",

scripts/lib/package-compatibility-projection.js

@@ -43,6 +43,7 @@ function walkFiles (root) {
 function assertSafeTarget (relative) {
   const value = portable(relative)
   const safe = value === 'instructions.md' ||
+    value === 'skills/portfolio.json' ||
     /^instructions\/[^/].+$/.test(value) ||
     /^prompts\/[^/].*\.md$/.test(value) ||
     /^skills\/(?:_schemas\/[^/].+|[^/]+\/[^/].*)$/.test(value)
@@ -66,7 +67,7 @@ function buildPackageProjectionPlan (root) {
   for (const file of walkFiles(contentSkillsRoot)) {
     const relative = portable(path.relative(contentSkillsRoot, file))
     if (relative.endsWith('/SKILL.md') || relative.endsWith('/devcodex.skill.json')) continue
-    if (relative === 'portfolio.json' || relative === 'portfolio-evidence.json') continue
+    if (relative === 'portfolio-evidence.json') continue
     const target = assertSafeTarget(`skills/${relative}`)
     if (entries.has(target)) throw new Error(`duplicate package projection target: ${target}`)
     const content = fs.readFileSync(file)

scripts/test-global-install-smoke.js

@@ -122,6 +122,11 @@ for (const host of ['.copilot', '.claude', '.codex', path.join('gemini-cli-home'
   assert.strictEqual(receipt.workspaceCleanMode, 'GlobalOnlyWorkspaceCleanModeV1')
 }
 assert.strictEqual(fs.existsSync(path.join(globalHome, '.agents', 'devcodex', 'instructions.full.md')), true)
+assert.strictEqual(
+  fs.existsSync(path.join(globalHome, '.agents', 'devcodex', 'skills', 'portfolio.json')),
+  true,
+  'shared Skill runtime must include portfolio.json for installed SkillRoute bootstrap'
+)
 assert.strictEqual(fs.existsSync(path.join(globalHome, '.agents', 'devcodex', 'skills', 'routing', 'SKILL.md')), true)
 assert.strictEqual(
   fs.existsSync(path.join(globalHome, '.agents', 'skills', 'routing', 'SKILL.md')),
@@ -219,6 +224,33 @@ const importedClaudeHook = JSON.parse(runCommand(process.execPath, [installedCla
 assert.strictEqual(importedClaudeHook.continue, true)
 assert.strictEqual(importedClaudeHook.devcodexCompatibilityBypass, 'grok-imported-claude-hook')
 
+const installedCodexAdapter = path.join(
+  globalHome,
+  '.codex',
+  'devcodex',
+  'runtime',
+  'hooks',
+  '_runtime',
+  'lifecycle-host-adapters.cjs'
+)
+const installedCodexHook = JSON.parse(runCommand(process.execPath, [installedCodexAdapter, 'codex'], {
+  cwd: workspace,
+  env: installedEnv,
+  shell: false,
+  input: JSON.stringify({
+    hookEventName: 'UserPromptSubmit',
+    cwd: workspace,
+    prompt: 'routing installed package smoke',
+    session_id: 'installed-package-skill-route'
+  })
+}).stdout)
+const installedCodexContext = `${installedCodexHook.systemMessage || ''}\n${installedCodexHook.hookSpecificOutput?.additionalContext || ''}`
+assert.match(
+  installedCodexContext,
+  /SkillRouteBootstrapV1/,
+  'installed Codex hook must expose SkillRoute bootstrap from packed global runtime'
+)
+
 for (const host of ['copilot', 'claude', 'codex', 'gemini', 'grok']) {
   const installedAdapter = path.join(
     globalHome,

scripts/test-pack-clean.js

@@ -53,7 +53,6 @@ const forbidden = [
   /data\/process-improvements\.md/,
   /data\/pending-issues\.md/,
   /data\/gap-registry\.md/,
-  /skills\/portfolio\.json/,
   /skills\/portfolio-evidence\.json/,
   /content-source\//,
   /^content\//,
@@ -106,6 +105,7 @@ const required = [
   'scripts/instruction-fallback-check.js',
   'scripts/migrate-layout.js',
   'assets/icon-512.png',
+  'skills/portfolio.json',
 ].concat(packageFiles, pluginFiles, promptFiles, dataTemplateFiles, indexRuntimeRequires, packagedScriptDeps)
   .filter(Boolean)
   .filter(file => !file.endsWith('/'))
@@ -131,8 +131,8 @@ if (skillPackFiles.length < 10) {
   console.error(`  found ${skillPackFiles.length} skills/*/SKILL.md`)
   process.exit(1)
 }
-if (files.some(file => file === 'skills/portfolio.json' || file === 'skills/portfolio-evidence.json')) {
-  console.error('\x1b[31m✗ Pack must exclude skills/portfolio.json and skills/portfolio-evidence.json\x1b[0m')
+if (files.some(file => file === 'skills/portfolio-evidence.json')) {
+  console.error('\x1b[31m✗ Pack must exclude skills/portfolio-evidence.json\x1b[0m')
   process.exit(1)
 }
 if (npmignore.includes('tests/')) {

scripts/test-package-compatibility-projection.js

@@ -19,7 +19,7 @@ const {
 } = require('./lib/package-compatibility-projection')
 
 const ROOT = path.resolve(__dirname, '..')
-const EXPECTED_ENTRY_COUNT = 249
+const EXPECTED_ENTRY_COUNT = 250
 
 function git (root, args) {
   return execFileSync('git', ['-C', root, ...args], {
@@ -93,6 +93,7 @@ if (currentMissingCount === 0 && currentReusableForeignCount === 0) {
     assert.strictEqual(materialized.materializedCount, EXPECTED_ENTRY_COUNT)
     assert.strictEqual(materialized.reusedExistingCount, 0)
     assert.strictEqual(fs.existsSync(path.join(root, 'instructions.md')), true)
+    assert.strictEqual(fs.existsSync(path.join(root, 'skills', 'portfolio.json')), true)
     assert.strictEqual(fs.existsSync(path.join(root, 'skills', 'routing', 'intent.json')), true)
     assert.strictEqual(fs.existsSync(path.join(root, DEFAULT_RECEIPT)), true)