Skip to content

feat(feishu): implement #308 #172 #307#9

Open
aaronwong1989 wants to merge 1 commit into
mainfrom
feat/feishu-all-issues
Open

feat(feishu): implement #308 #172 #307#9
aaronwong1989 wants to merge 1 commit into
mainfrom
feat/feishu-all-issues

Conversation

@aaronwong1989
Copy link
Copy Markdown
Owner

Implement Feishu streaming card API and related features.

Changes

  • card_api.go: Add CreateCard/UpdateCard/SendCardMessage APIs with retry logic
  • card_api_test.go: Comprehensive test coverage for card APIs
  • adapter.go: Add adapter functions
  • converter.go: Add message converter
  • streaming.go: Update streaming implementation
  • client.go: Add client support

Related Issues

- hrygo#308: Add CardKit API exponential backoff retry (SendCardWithRetry)
  * 3 retries, 100ms initial delay, 5s max delay
  * Retry on network errors and 5xx, no retry on 4xx
  * Used in streaming.go for card message sending

- hrygo#172: Integrate ProcessorChain into Feishu adapter
  * Add processorChain field to Adapter struct
  * Initialize with NewDefaultProcessorChain using FeishuConverter
  * Route defaultSender through processor chain before sending
  * Create FeishuConverter for Markdown -> Feishu format conversion

- hrygo#307: WebSocket and CardKit test coverage improvement
  * Add CardKit retry logic tests (isRetryableError, calculateBackoff)
  * Add WebSocket reconnect/handler tests
  * Add streaming writer state tests
  * Add processor chain integration tests
  * Coverage: 39.3% -> 46.8%

- Fix MockFeishuClient to implement SendCardWithRetry interface
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