fix(typefully): tag-only updates + safer aliases; add CLI tests#8
Merged
fix(typefully): tag-only updates + safer aliases; add CLI tests#8
Conversation
- Support tag-only drafts:update without modifying content\n- Harden create-draft/update-draft aliases (clean errors on missing values)\n- Honor --social-set-id/--social_set_id in config:set-default + other commands\n- Improve thread splitting to handle CRLF\n- Add hermetic node:test suite + GitHub Actions CI
Document changes in this branch (CLI fixes, new tests/CI).
- Move changelog into skill folder (skills/typefully/CHANGELOG.md)\n- Document that changelog entries must be user-facing only\n- Instruct agents to update the relevant skill changelog when changing CLI/skill behavior
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes a bug where
update-draft <id> --tags ... --social-set-id ...could overwrite the draft content with the literal flags/values.Also adds hermetic CLI tests + CI so these edge cases stay covered.
Changes
update-draftalias to only forward--textwhen text is actually provided (flag or positional), so tag-only updates don't touch content.--tagssupport todrafts:update(including clearing tags with--tags "").config:set-defaulthonor--social-set-id/--social_set_id(help NOTE is now accurate).---.node --testsuite (mock HTTP server, hermetic HOME/cwd) and GitHub Actions matrix (Node 18/20/22).Repro
typefully create-draft "Test content" --social-set-id 100812typefully update-draft <id> --tags "owner/me" --social-set-id 100813Expected: tags added, content unchanged.
Actual (before): content replaced by flags as text.
Testing
node --test