Skip to content

Add current Xquik Skill and fix seed build failures - #13

Open
kriptoburak wants to merge 1 commit into
nextlevelbuilder:mainfrom
kriptoburak:codex/add-xquik-skillx-entry
Open

Add current Xquik Skill and fix seed build failures#13
kriptoburak wants to merge 1 commit into
nextlevelbuilder:mainfrom
kriptoburak:codex/add-xquik-skillx-entry

Conversation

@kriptoburak

@kriptoburak kriptoburak commented May 8, 2026

Copy link
Copy Markdown

Summary

  • Add the current Xquik x-twitter-scraper Skill to the SkillX seed catalog.
  • Cover REST, remote MCP, SDKs, search, exports, monitoring, webhooks, and approval-gated publishing.
  • Use the public Skill source and current install command.
  • Keep read-only defaults, explicit approval gates, untrusted-content handling, and credential boundaries.
  • Remove invented install and rating metrics from the new seed entry.
  • Make seed-build failures return a nonzero process status.
  • Add regression coverage for unreadable seed data.

Independent Fix

build-seed-data.mjs previously used main().catch(console.error). Read, parse, or write failures printed an error but left the process exit status at 0. Automation could therefore accept a failed seed build. The script now sets process.exitCode = 1, and the regression test executes an isolated copy without seed-data.json to verify the failure contract.

Validation

  • pnpm install --frozen-lockfile
  • pnpm test (39 tests pass)
  • pnpm build
  • node --check scripts/build-seed-data.mjs
  • node --check scripts/build-seed-data.test.mjs
  • node scripts/build-seed-data.mjs
  • Parsed scripts/seed-data.json and verified exactly 1 Xquik row.
  • Verified the public Skill source, MCP endpoint, and OpenAPI endpoint.
  • git diff --check

pnpm typecheck still reports pre-existing errors in unchanged app files. This PR adds no TypeScript application code and introduces no new typecheck error.

@kriptoburak
kriptoburak force-pushed the codex/add-xquik-skillx-entry branch from 397ca3f to 060a8e5 Compare July 30, 2026 03:45
@kriptoburak kriptoburak changed the title Add Xquik x-twitter-scraper skill Add current Xquik Skill and fix seed build failures Jul 30, 2026
@kriptoburak

Copy link
Copy Markdown
Author

@codex review

Please review the refreshed Xquik seed entry, removal of synthetic engagement metrics, and the independent nonzero-exit fix for failed seed builds. Commit 060a8e5 passes all 39 tests and the production build.

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