Skip to content

Add aident-skill#120

Open
lukeguo12210 wants to merge 1 commit into
daymade:mainfrom
lukeguo12210:skill/aident-skill
Open

Add aident-skill#120
lukeguo12210 wants to merge 1 commit into
daymade:mainfrom
lukeguo12210:skill/aident-skill

Conversation

@lukeguo12210

Copy link
Copy Markdown

Summary

  • add the aident-skill package to the marketplace
  • register it in .claude-plugin/marketplace.json
  • add a README listing and bump the marketplace count/version

Validation

  • git diff --check
  • parsed .claude-plugin/marketplace.json with Node.js

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 13465db5ed

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread aident-skill/SKILL.md
---
name: aident-skill
description: Use Aident Loadout to connect your AI agents to 1,000+ real-world apps and tools like Gmail, Slack, Linear, Notion, Firecrawl, and Fal, unlock 27,000+ executable actions, and track full audit history so your agents can get real work done reliably.
author: Aident

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Remove unsupported SKILL.md frontmatter

The repo's packaged validator (daymade-skill/skill-creator/scripts/quick_validate.py) rejects unexpected SKILL.md frontmatter keys, and package_skill.py imports that validator before packaging; starting here this file adds unsupported keys such as author, homepage, repository, tags, categories, x-*, and version. This newly registered skill cannot pass the required validation/package workflow until those metadata fields are removed or moved into supported marketplace metadata.

Useful? React with 👍 / 👎.

Comment thread README.md

## 📦 Other Available Skills

### **aident-skill** - Aident Loadout For Agent Integrations

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Sync the required numbered skill lists

This new README entry is not in the numbered list form that the repo's drift guard parses (### N. **name**), and the required CLAUDE.md / README.zh-CN updates are missing; running python3 daymade-claude-code/marketplace-dev/scripts/check_doc_skill_lists.py reports aident-skill missing from CLAUDE.md, README.md, and README.zh-CN, plus the zh-CN version badge still at 1.78.0. As a result the documented marketplace check fails until all required docs are synced.

Useful? React with 👍 / 👎.

Comment thread aident-skill/README.md
Comment on lines +11 to +12
<a href="https://skills.sh/aident-ai/aident-skill"><img src="https://img.shields.io/badge/skills.sh-aident--skill-blue" alt="Skills.sh"></a>
<a href="https://clawhub.ai/aident-ai/skills/aident-skill"><img src="https://img.shields.io/badge/ClawHub-aident--skill-111827" alt="ClawHub"></a>

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Drop third-party marketplace promotion links

I checked references/promotion-policy.md; it says this repo is a personal curated marketplace and not to accept third-party marketplace links or skill collection references. These Skills.sh/ClawHub badges, together with the install table below, add third-party marketplace promotion inside this repo, so the contribution violates that repo policy and should be removed or declined.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant