Skip to content

Add Xquik research skill and fix validator line limit - #2

Open
kriptoburak wants to merge 1 commit into
crystian:mainfrom
kriptoburak:codex/add-xquik-research-skill
Open

Add Xquik research skill and fix validator line limit#2
kriptoburak wants to merge 1 commit into
crystian:mainfrom
kriptoburak:codex/add-xquik-research-skill

Conversation

@kriptoburak

@kriptoburak kriptoburak commented Jul 3, 2026

Copy link
Copy Markdown

Type

  • New skill
  • Bug fix
  • Tooling / CI

Summary

  • Add xquik-x-research for evidence-backed X audience and market research.
  • Document canonical Xquik source, REST, OpenAPI, and MCP references.
  • Correct the REST base URL and replace a dead API documentation link.
  • Fix validator body-line counting at the no-trailing-newline boundary.
  • Add 500-line and 501-line regression tests and run them in CI.

Independent Repository Fix

The validator used body.count("\n"), so a 501-line body without a trailing newline was counted as 500 and incorrectly accepted. It now uses splitlines(), accepts exactly 500 lines, and rejects 501 lines regardless of the final newline.

Checklist

  • Read CONTRIBUTING.md and CLAUDE.md.
  • One skill plus one focused validator boundary fix.
  • Required frontmatter fields are present.
  • Body is below 500 lines.
  • Directory name matches name.
  • User-facing README is current.
  • No secrets, absolute local paths, or personal data.
  • Version remains 0.1.0 for the new skill.

Validation

  • python3 scripts/validate-skills.py
  • python3 -m unittest scripts/test_validate_skills.py
  • python3 -m py_compile scripts/validate-skills.py scripts/test_validate_skills.py
  • actionlint .github/workflows/validate-skills.yml
  • Markdown link checks for root and Xquik docs
  • git diff --check
  • Disclosure, punctuation, and secret-shape scans

Xquik is an independent third-party service. Not affiliated with X Corp. "Twitter" and "X" are trademarks of X Corp.

@kriptoburak kriptoburak changed the title Add Xquik X research skill Add Xquik research skill and fix validator line limit Jul 18, 2026
@kriptoburak

kriptoburak commented Jul 18, 2026

Copy link
Copy Markdown
Author

Published current-upstream repair c252840e2aaed71f744678768ba0e74d64d81479 as 1 signed commit.

The Xquik research skill uses the canonical source, live API and MCP documentation, correct /api/v1 base, current authentication guidance, and required disclosure. The independent repository fix corrects validator line undercounting for bodies without a trailing newline, adds exact 500 and 501-line regression tests, and runs them in CI.

Validation passed for all repository skills, both unit tests, Python compilation, workflow YAML, all 5 live Xquik references, whitespace, and the current upstream diff. No contributor action remains.

@kriptoburak
kriptoburak force-pushed the codex/add-xquik-research-skill branch from c9dfc5f to c252840 Compare July 19, 2026 20:38
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