Skip to content

skills: Clarify public API sync output - #216

Merged
mguida22 merged 3 commits into
mainfrom
guida/api-sync-public-artifacts
Nov 7, 2026
Merged

mguida22 merged 3 commits into
mainfrom
guida/api-sync-public-artifacts

Conversation

@mguida22

@mguida22 mguida22 commented Nov 6, 2026 •

Copy link
Copy Markdown
Member

Changelog

None

Docs

None

Description

The API sync skill prohibits private implementation details in public output, but its source-link and revision instructions leave that boundary ambiguous. Clarify that public artifacts should describe public API and Python client behavior, allow public references and Foxglove Linear issue links while excluding other non-public URLs, and keep private source revisions out of branch names and public output. Add a final check for internal details before delivery.

Validation: skill validation and git diff --check passed. No runtime behavior changes.

BeforeAfter

Source-link and revision instructions could conflict with the public-content rule.

Public references and behavior descriptions are required, with an exception for Linear issue IDs and short links used for tracking and issue closure; private source revisions remain in the private conversation.

BFCK-588

[ GENERATED BY AI ]

@linear-code

linear-code Bot commented Nov 6, 2026

Copy link
Copy Markdown

BFCK-588

foxglove-beefcake[bot]

This comment was marked as outdated.

@foxglove-beefcake foxglove-beefcake 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.

LGTM

Risk: low. The change edits only agent skill documentation and does not change runtime code or the published package.

@foxglove-beefcake foxglove-beefcake 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.

LGTM

Risk: low. The change edits only agent skill documentation and does not change runtime code or the published package.

@mguida22
mguida22 merged commit b7bc41f into main Nov 7, 2026
2 checks passed
@mguida22
mguida22 deleted the guida/api-sync-public-artifacts branch November 7, 2026 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants