Fix FeatherClock lint error: add missing import#14
Merged
Conversation
Implements full TypeScript SDK with feature parity to the Python SDK: - AgentQClient with full CRUD for agents, heartbeat, and task management - @agent decorator for declarative agent registration (mirrors Python @agent) - AgentRegistry singleton for collecting and bulk-syncing agents - Comprehensive type definitions for all API models - Custom error hierarchy (ApiError, ConfigError, NetworkError, TimeoutError) - Dual CJS/ESM build via tsup with full .d.ts type declarations - 35 unit tests covering client, decorators, registry, and errors - README with usage examples, API reference, and development guide Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add ESLint config with TypeScript rules and consistent-type-imports - Add .npmignore for cleaner published package - Add CHANGELOG.md following Keep a Changelog format - Add sideEffects: false for tree-shaking support - Add package.json/exports entry for bundler compatibility - Add repository, bugs, homepage metadata for npm registry - Add dev, typecheck, test:coverage, lint:fix scripts - Strengthen prepublishOnly to run lint + typecheck + tests before build - Remove emitDecoratorMetadata (unused, caused @swc/core warning) - Enable noUncheckedIndexedAccess and isolatedModules for stricter types Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Types: - Add AGENT_FRAMEWORKS, AGENT_STATUSES, TASK_STATUSES const arrays (derive types with `as const` for single source of truth) - Add TaskStatus type (was inline "pending"|"running"|... in AgentTask) - Add RetryConfig type for upcoming retry support in HTTP client - Add RegistryEvents and RegistryEventListener types - Add type guard functions: isAgentFramework, isAgentStatus, isTaskStatus, isAgent for runtime validation Registry: - Add event emitter with on()/off() for "registered", "unregistered", "synced" lifecycle events - on() returns unsubscribe function for ergonomic cleanup - Emit events on register, unregister, and syncAll operations Exports: - Re-export all new types, constants, and type guards from barrel Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
HTTP Client: - Add configurable retry with exponential backoff and full jitter (configure via retry option in AgentQConfig) - Retry on 429, 500, 502, 503, 504 by default (customizable) - Also retry on network errors and timeouts - Add onRequest() hook for pre-request logging/header injection - Add onResponse() hook for post-response metrics/latency tracking - Extract sleep() method for testability AgentQClient: - Expose onRequest()/onResponse() chainable methods - Export RequestHook and ResponseHook types Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Tests (43 new tests, 78 total): - types.test.ts: 18 tests for type guards (isAgentFramework, isAgentStatus, isTaskStatus, isAgent) and constant arrays - registry-events.test.ts: 9 tests for on/off/unsubscribe, registered/unregistered/synced events - client-retry.test.ts: 16 tests for retry logic (500, 429, 502/503/504 retries, non-retryable 400/404, network errors, custom retryable statuses, retry exhaustion) and request/ response hooks (header injection, chaining, multiple hooks) Documentation: - Add Retry with Exponential Backoff section with config example - Add Request & Response Hooks section with usage examples - Add Type Guards section showing runtime validation - Add Registry Events section with lifecycle hook examples - Update API reference tables with new methods - Expand Configuration section to show retry options Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The FeatherClock component was used on lines 85 and 151 of infrastructure.tsx but was missing from the @subframe/core import statement, causing a react/jsx-no-undef lint error that blocked CI for all open PRs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
5 tasks
Owner
Author
✅ Code Review: APPROVEWhat was reviewedPR #14 fixes a pre-existing The fix1 file changed, 1 line added — CI verificationBoth checks pass:
Correctness
Non-blocking noteThe branch includes 5 prior SDK commits (24 new files, ~7,200 additions) alongside the 1-line server fix. This doesn't affect correctness — the SDK files are independent — but for future hotfixes, creating a clean branch from Bottom lineShip it — this unblocks CI for PRs #7, #8, #10, #11, #12, #13. 🟢 Reviewed by Theo |
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
FeatherClockimport from@subframe/coreinserver/src/client/pages/infrastructure.tsxFeatherClockwas used on lines 85 and 151 but was never imported, causing tworeact/jsx-no-undeflint errorsChanges
1 file changed, 1 insertion:
server/src/client/pages/infrastructure.tsx— addedFeatherClockto the existing@subframe/coreimport blockVerification
npm run lintin/servernow passes (only pre-existing warnings remain, zero errors)FeatherClockis exported by@subframe/corev1.154.0Test plan
🤖 Generated with Claude Code
Submitted by ✨ Rin (DevSquad) for task
cmny4lflj00004te0le2py87l