Skip to content

Fix type assertions in work.ts#81

Open
tembo[bot] wants to merge 1 commit intomainfrom
tembo/shippost-scannerbot-fix-1-1
Open

Fix type assertions in work.ts#81
tembo[bot] wants to merge 1 commit intomainfrom
tembo/shippost-scannerbot-fix-1-1

Conversation

@tembo
Copy link
Copy Markdown
Contributor

@tembo tembo bot commented Apr 11, 2026

Summary

Replaced any type assertions with proper LLMService type in work.ts:

  • Updated generateBlogDraft() function parameter from llm: any to llm: LLMService
  • Updated updateRelatedBlogPosts() function parameter from llm: any to llm: LLMService
  • Added import for LLMService type from ../services/llm-service.js

Also created two new code quality tasks in .beads/issues.jsonl:

  • t2p-cq09: Fix as any type assertions in reply.ts for Tweet with parentTweetForContext
  • t2p-cq10: Review empty catch blocks in blog.ts for proper error handling

Want tembo to make any changes? Add a review or comment with @tembo and i'll get back to work!

View on Tembo View Agent Settings View on automation

@tembo tembo bot added the tembo Pull request created by Tembo label Apr 11, 2026
@tembo
Copy link
Copy Markdown
Contributor Author

tembo bot commented Apr 11, 2026

Requesting review from @ryw who has experience with the following files modified in this PR:

  • .beads/issues.jsonl
  • src/commands/work.ts

@tembo tembo bot requested a review from ryw April 11, 2026 08:03
@blocksorg
Copy link
Copy Markdown

blocksorg bot commented Apr 11, 2026

Mention Blocks like a regular teammate with your question or request:

@blocks review this pull request
@blocks make the following changes ...
@blocks create an issue from what was mentioned in the following comment ...
@blocks explain the following code ...
@blocks are there any security or performance concerns?

Run @blocks /help for more information.

Workspace settings | Disable this message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tembo Pull request created by Tembo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants