Skip to content

chore: remove internal signer Slack skill - #5

Merged
0xhafa merged 1 commit into
mainfrom
chore/remove-signer-slack-skill
Sep 21, 2026
Merged

0xhafa merged 1 commit into
mainfrom
chore/remove-signer-slack-skill

Conversation

@0xhafa

@0xhafa 0xhafa commented Sep 21, 2026

Copy link
Copy Markdown
Collaborator

Remove the internal signer Slack-message drafting skill from the public repository. The grammar check currently requires that file, so remove its skill-specific validations while preserving the shared grammar reference's SHA-256 and version checks.

Validation:

  • Reproduced the missing-skill CI failure after deleting the directory.
  • pnpm check:signer-grammar, node --check, and git diff --check pass.
  • Temporary fixtures confirm the checker still rejects a missing grammar reference, changed bytes, and an incorrect grammar version.
  • No remaining references to the removed skill. No contract or grammar-reference changes.

This cleanup targets public main; the private repository's internal skill is unchanged.

@0xhafa
0xhafa merged commit d7e1f89 into main Sep 21, 2026
3 checks passed
@greptile-apps

greptile-apps Bot commented Sep 21, 2026

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable regressions were identified.

Summary

Removes the internal signer Slack-message drafting skill from the public repository.

  • Narrows the grammar checker to the retained public grammar reference.
  • Preserves missing-reference, SHA-256 integrity, and grammar-version checks.
  • Removes the obsolete success message claiming agreement with the deleted generator skill.

Reviews (1) · Last reviewed commit: "chore: remove internal signer Slack skil..."

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