Skip to content

docs: AI standardization from tibblify#15

Merged
jonthegeek merged 4 commits into
mainfrom
copilot/adapt-agents-docs
Mar 9, 2026
Merged

docs: AI standardization from tibblify#15
jonthegeek merged 4 commits into
mainfrom
copilot/adapt-agents-docs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 9, 2026

Brings AGENTS.md and .github/skills/ in line with wranglezone/tibblify, adapting for datawrap's pure-R stack (no C layer).

AGENTS.md

  • Standard workflow (steps 1–9): structured dev loop — update packages → test-first → implement → refactor → document → verify → NEWS
  • General section: --quiet --vanilla, air format ., comment philosophy, skill loading reminder
  • Skills table: expanded to reference three new skills below

New skills

  • implement-issue/SKILL.md: end-to-end issue workflow — branch, standard workflow, commit/PR
  • r-code/SKILL.md: API design patterns, internal vs exported, error handling, dependency policy — stripped of tibblify-specific tib_*/tspec_* conventions
  • tdd-workflow/SKILL.md: TDD loop, coverage checks (vapply per repo convention), snapshot patterns, deprecated testthat idioms — datawrap examples throughout
Original prompt

This section details on the original issue you should resolve

<issue_title>docs: ai standardization</issue_title>
<issue_description>Adapt the AGENTS.md and .github/skills in wranglezone/tibblify to this package. {tibblify} uses C while this package does not, so we should remove those references.</issue_description>

<agent_instructions>Use these files from tibblify as a guide, as well as our existing AGENTS.md and .github/skills:

For any existing skills in datawrap, compare against the equivalent in tibblify.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: jonthegeek <33983824+jonthegeek@users.noreply.github.com>
Copilot AI changed the title [WIP] Update AGENTS.md and skills for standardization docs: AI standardization from tibblify Mar 9, 2026
@jonthegeek jonthegeek marked this pull request as ready for review March 9, 2026 12:56
Copilot AI review requested due to automatic review settings March 9, 2026 12:56
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s agent-facing documentation to standardize development workflow guidance and add new skill modules adapted from wranglezone/tibblify, tailored to datawrap’s pure-R setup.

Changes:

  • Adds a “Standard workflow” section to AGENTS.md and refreshes general guidance + skills index.
  • Introduces three new skills: implement-issue, r-code, and tdd-workflow.
  • Updates .Rbuildignore to ignore a .claude path.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
AGENTS.md Adds a structured standard workflow section and refreshes skills + general guidance.
.github/skills/tdd-workflow/SKILL.md New TDD/testing/coverage guidance and examples for datawrap.
.github/skills/r-code/SKILL.md New API design + dependency/error-handling guidance for R code in this package.
.github/skills/implement-issue/SKILL.md New end-to-end “implement an issue” workflow wrapping the standard workflow.
.Rbuildignore Ignores .claude from package builds.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/skills/r-code/SKILL.md
Comment thread .github/skills/implement-issue/SKILL.md
Comment thread .github/skills/tdd-workflow/SKILL.md
Comment thread .github/skills/tdd-workflow/SKILL.md Outdated
Comment thread .github/skills/tdd-workflow/SKILL.md Outdated
Comment thread .github/skills/tdd-workflow/SKILL.md
Comment thread .github/skills/r-code/SKILL.md
Use more realistic example.
Remove irrelevant integration test example.
@jonthegeek jonthegeek enabled auto-merge (squash) March 9, 2026 13:20
@jonthegeek jonthegeek merged commit 46fd23b into main Mar 9, 2026
7 checks passed
@jonthegeek jonthegeek deleted the copilot/adapt-agents-docs branch March 9, 2026 13:21
Copilot AI added a commit that referenced this pull request Mar 9, 2026
- Add .github/workflows/install/action.yml composite action for shared
  R/pandoc/dependency setup
- Add .github/workflows/copilot-setup-steps.yml for GitHub Copilot agent
- Update R-CMD-check, pkgdown, test-coverage, pr-commands to use install action
- Switch /style pr-command from styler to air format

- Closes #15

Co-authored-by: jonthegeek <33983824+jonthegeek@users.noreply.github.com>
jonthegeek added a commit that referenced this pull request Mar 10, 2026
* Initial plan

* ci: add copilot setup workflow and componentize install steps

- Add .github/workflows/install/action.yml composite action for shared
  R/pandoc/dependency setup
- Add .github/workflows/copilot-setup-steps.yml for GitHub Copilot agent
- Update R-CMD-check, pkgdown, test-coverage, pr-commands to use install action
- Switch /style pr-command from styler to air format

- Closes #15

Co-authored-by: jonthegeek <33983824+jonthegeek@users.noreply.github.com>

* fix: checkout version in copilot setup workflow

* Try to provide codecov token.

* Another possible way to provide the token

* Try another token path.

* Match usethis.

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: jonthegeek <33983824+jonthegeek@users.noreply.github.com>
Co-authored-by: Jon Harmon <jonthegeek@gmail.com>
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.

docs: ai standardization

3 participants