fix: add displayName, description, and icon to marketplace entries (#93)

* fix: enrich marketplace entries with displayName, description, and icon

- Add displayName from README plugin names to marketplace entries
- Add description from README entries (with manifest shortDescription fallback)
- Add icon path extracted from plugin manifest's interface.composerIcon/logo
- Ensure icons are correctly resolved relative to marketplace.json
- Fixes issue where all plugin titles showed as 'awesome-codex-plugins'
- Fixes missing images and descriptions in Codex marketplace view

* fix: exclude icon entries when asset file is missing

Some plugins declare composerIcon/logo paths in their manifest but the
corresponding files are not present in the repo (e.g., placeholder values or
missing assets). The marketplace now verifies the file exists before including
the icon reference, avoiding broken image links in Codex.

* fix: correct icon path formatting (remove duplicate ./ prefix) and filter missing assets

 marketplace.json entries now include icon only when the declared asset file
actually exists in the mirrored plugin directory. This prevents broken image
references for plugins that have placeholder values or missing files.

* fix: filter placeholder icon values and clear plugin destinations before mirroring

- Thread 1 fix: reject placeholder icon values like "[TODO: ./assets/icon.png]"
  by checking for bracketed TODO/PLACEHOLDER patterns before including icon
- Thread 2 fix: clear destination_root in mirror_plugin_bundle() to avoid
  stale files from previous runs causing false-positive icon existence checks

Both changes address chatgpt-codex-connector review feedback.

* chore: sync plugin artifacts after mirror fix

Update mirrored plugin bundles to reflect current upstream selections;
clean stale files from previous runs. This resolves the "Sync marketplace
artifacts with README" CI failure by ensuring working tree matches
generator output after the rmtree fix.

* fix: preserve dot-prefixed paths during normalization (P1 regression)

normalize_relative_path() previously used lstack("./") which stripped all
leading '.' and '/' characters, mangling dotfile paths like ".mcp.json"
into "mcp.json". This caused required dot-prefixed manifest files to be
omitted from mirrored plugin bundles when the destination was cleaned.

The function now only strips a literal leading "./" or "/" prefix, keeping
dotfiles intact. The manifest path is also normalized the same way when
computing the icon file path to ensure consistency.

Fixes chatgpt-codex-connector P1 review thread at line 221.

* fix: include hooks in selected_paths to preserve manifest-declared hook configs

chatgpt-codex-connector P1: mirror_plugin_bundle with rmtree now relies on
collect_selected_paths to fully reconstruct the plugin. The hooks field from
the manifest was not being copied, causing installed plugins to lose their
hook workflows. Added "hooks" to the manifest keys processed by
collect_selected_paths.

Restores:
- plugins/Kanevry/session-orchestrator/hooks/hooks-codex.json
- plugins/BestLemoon/codex-seo/hooks/hooks.json
- plugins/sendbird/cc-plugin-codex/hooks/hooks.json

* fix: commit script fixes (normalize_relative_path, hooks, icon path) and CI workflow guard

The previous commits included regenerated artifacts but missed committing
the script changes that produced them. This caused CI to run the old
(buggy) script against the new artifacts, detecting a mismatch.

Script fixes:
- normalize_relative_path: use startswith checks instead of lstrip to
  preserve dot-prefixed paths like .mcp.json
- collect_selected_paths: include 'hooks' manifest key
- icon path: use startswith checks instead of lstrip for consistent prefix
  stripping

Workflow fix:
- Skip git push on pull_request events (detached HEAD checkout)
- Auto-commit/push only on push to main

Author: internet-dot

.agents/plugins/marketplace.json

@@ -36,12 +36,17 @@ jobs:
         run: |
           python3 scripts/generate_plugins_json.py
           if ! git diff --quiet -- plugins.json .agents/plugins/marketplace.json plugins; then
-            echo "::notice::marketplace artifacts were out of sync — auto-committing"
-            git config user.name "github-actions[bot]"
-            git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
-            git add plugins.json .agents/plugins/marketplace.json plugins
-            git commit -m "chore: sync marketplace artifacts with README"
-            git push
+            echo "::notice::marketplace artifacts were out of sync"
+            if [ "${{ github.event_name }}" = "push" ]; then
+              echo "::notice::auto-committing synced artifacts"
+              git config user.name "github-actions[bot]"
+              git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
+              git add plugins.json .agents/plugins/marketplace.json plugins
+              git commit -m "chore: sync marketplace artifacts with README"
+              git push
+            else
+              echo "::notice::skipping push on ${{ github.event_name }} (detached HEAD on PR merge ref)"
+            fi
           else
             echo "marketplace artifacts are up to date"
           fi

.github/workflows/validate-plugins.yml

@@ -0,0 +1,3 @@
+{
+  "apps": {}
+}

plugins/6Delta9/task-scheduler-codex-plugin/.app.json

@@ -0,0 +1,16 @@
+{
+  "mcpServers": {
+    "taskScheduler": {
+      "command": "python",
+      "args": [
+        "./scripts/mcp_server.py"
+      ],
+      "cwd": ".",
+      "env": {
+        "PYTHONUTF8": "1"
+      },
+      "startup_timeout_sec": 20,
+      "tool_timeout_sec": 60
+    }
+  }
+}

plugins/6Delta9/task-scheduler-codex-plugin/.mcp.json

@@ -0,0 +1,3 @@
+{
+  "hooks": []
+}

plugins/6Delta9/task-scheduler-codex-plugin/hooks.json

@@ -0,0 +1,15 @@
+{
+  "hooks": {
+    "PostToolUse": [
+      {
+        "matcher": "Write|Edit",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "py -3 ./scripts/post_write_codex_agent_guardrail.py || python ./scripts/post_write_codex_agent_guardrail.py || python3 ./scripts/post_write_codex_agent_guardrail.py"
+          }
+        ]
+      }
+    ]
+  }
+}

plugins/AlexMi64/codex-project-autopilot/hooks.json

@@ -0,0 +1,37 @@
+{
+  "mcpServers": {
+    "dataforseo": {
+      "command": "npx",
+      "args": [
+        "-y",
+        "dataforseo-mcp-server"
+      ],
+      "env": {
+        "DATAFORSEO_USERNAME": "${DATAFORSEO_USERNAME}",
+        "DATAFORSEO_PASSWORD": "${DATAFORSEO_PASSWORD}",
+        "ENABLED_MODULES": "SERP,KEYWORDS_DATA,ONPAGE,DATAFORSEO_LABS,BACKLINKS,DOMAIN_ANALYTICS,BUSINESS_DATA,CONTENT_ANALYSIS,AI_OPTIMIZATION",
+        "FIELD_CONFIG_PATH": "./extensions/dataforseo/field-config.json"
+      }
+    },
+    "firecrawl-mcp": {
+      "command": "npx",
+      "args": [
+        "-y",
+        "firecrawl-mcp"
+      ],
+      "env": {
+        "FIRECRAWL_API_KEY": "${FIRECRAWL_API_KEY}"
+      }
+    },
+    "nanobanana-mcp": {
+      "command": "npx",
+      "args": [
+        "-y",
+        "@ycse/nanobanana-mcp@latest"
+      ],
+      "env": {
+        "GOOGLE_AI_API_KEY": "${GOOGLE_AI_API_KEY}"
+      }
+    }
+  }
+}

plugins/BestLemoon/codex-seo/.mcp.json

@@ -0,0 +1,15 @@
+{
+  "hooks": {
+    "PostToolUse": [
+      {
+        "matcher": "Edit|Write",
+        "hooks": [
+          {
+            "type": "command",
+            "command": "python3 \"./hooks/validate-schema.py\" \"$FILE_PATH\""
+          }
+        ]
+      }
+    ]
+  }
+}

plugins/BestLemoon/codex-seo/hooks/hooks.json

@@ -0,0 +1,9 @@
+{
+  "mcpServers": {
+    "launchfast": {
+      "type": "http",
+      "url": "https://launchfastlegacyx.com/api/mcp/server",
+      "note": "LaunchFast MCP server for Amazon product research. Codex handles OAuth through the LaunchFast MCP metadata flow. During authorization, localhost or 127.0.0.1 loopback callback URLs are expected for the local Codex client and should not be rewritten to the LaunchFast domain."
+    }
+  }
+}