Repository navigation
Conversation
Bug fixes: - Case-insensitive section heading validation. Sonnet sometimes produces lowercase headings which caused valid summaries to fail and trigger retries. - Test env leak: SEAMLESS_DISPLAY_CMD no longer inherited from host environment in integration tests. Features: - SEAMLESS_INDICATOR env var for custom display commands. Shows S when idle, emoji when active. - Error logging for UPS hook (ups-errors.log). Community: - Apache 2.0 licence, contributing guide, security policy, issue/PR templates, changelog.
There was a problem hiding this comment.
Pull request overview
This PR releases version 0.1.1 with bug fixes, a new indicator feature for custom statuslines, improved error logging, and community contribution files.
Changes:
- Fixed case-insensitive section heading validation to prevent false rejections when Claude produces lowercase headings
- Added
SEAMLESS_INDICATORenvironment variable (always set toSor status emoji) for custom display commands - Added error logging to
~/.seamless-claude/ups-errors.logfor UserPromptSubmit hook failures - Fixed test environment leakage of
SEAMLESS_DISPLAY_CMD - Added community files (CONTRIBUTING.md, SECURITY.md, CHANGELOG.md, GitHub templates)
- Streamlined CI workflow to test Node 18 and 22 only
Reviewed changes
Copilot reviewed 18 out of 19 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| lib/validate.mjs | Implements case-insensitive section heading matching |
| test/validate.test.mjs | Adds test coverage for case-insensitive validation |
| lib/statusline.mjs | Adds indicator() function and STATUS_INDICATORS mapping |
| scripts/statusline.mjs | Passes SEAMLESS_INDICATOR to custom display commands |
| test/statusline.test.mjs | Adds indicator tests and fixes SEAMLESS_DISPLAY_CMD test leakage |
| scripts/user-prompt-submit.mjs | Adds error logging to ups-errors.log file |
| package.json | Version bump to 0.1.1 |
| package-lock.json | Version bump and bin field sync |
| .claude-plugin/plugin.json | Version bump to 0.1.1 |
| README.md | Documents indicator feature and error log location |
| CHANGELOG.md | New file documenting all changes in Keep a Changelog format |
| CONTRIBUTING.md | New contribution guide with setup, testing, and PR process |
| SECURITY.md | New security policy with vulnerability reporting instructions |
| LICENCE | Updates copyright from Edward Woodcock to RemoteCTO |
| NOTICE | Simplifies notice and updates copyright to RemoteCTO |
| .github/workflows/ci.yml | Streamlines workflow: Node 18/22 only, combines lint and test jobs |
| .github/PULL_REQUEST_TEMPLATE.md | New PR template with checklist including changelog requirement |
| .github/ISSUE_TEMPLATE/bug_report.md | New bug report template with environment and log sections |
| .github/ISSUE_TEMPLATE/feature_request.md | New feature request template |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Use npm ci in CI workflow (reproducible builds) - Fix lint:fix command in CONTRIBUTING.md (biome uses --write, not --fix) - Fix changelog: CI tests Node 18/22, not 18/20/22 - Use DATA_DIR for UPS error log path (respects SEAMLESS_DATA_DIR override)
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
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
session summaryvsSession Summary). The validator now matches case-insensitively, preventing valid summaries from being rejected and triggering unnecessary retries with halved transcripts.SEAMLESS_INDICATORenv var that's always set when the plugin is active (Swhen idle, emoji for other states). Makes it easy to confirm the plugin is running in your statusline.~/.seamless-claude/ups-errors.loginstead of being silently swallowed — aids diagnosis when Claude Code shows generic "hook error" messages.SEAMLESS_DISPLAY_CMDno longer leaks from host environment into integration tests.Test plan
Sindicator appears in statusline