Skip to content

Add schedule name correction for LLMs#4370

Merged
bfops merged 2 commits intoclockworklabs:masterfrom
Boegie19:schedule-syntax-correction
Feb 26, 2026
Merged

Add schedule name correction for LLMs#4370
bfops merged 2 commits intoclockworklabs:masterfrom
Boegie19:schedule-syntax-correction

Conversation

@Boegie19
Copy link
Contributor

Description of Changes

This change is needed since the AI halicunates that the name is schedule for some dam reason every time in the llm tests. while it is scheduled

"llm_output": "use spacetimedb::{table, reducer, ReducerContext, Table, ScheduleAt};\n\n#[table(name = tick_timer, schedule(reducer = tick, column = scheduled_at))]\npub struct TickTimer {\n #[primary_key]\n #[auto_inc]\n scheduled_id: u64,\n scheduled_at: ScheduleAt,\n}\n\n#[reducer(init)]\npub fn init(ctx: &ReducerContext) {\n if ctx.db.tick_timer().count() == 0 {\n ctx.db\n .tick_timer()\n .insert(TickTimer { scheduled_id: 0, scheduled_at: ScheduleAt::repeat_micros(50_000) });\n }\n}\n\n#[reducer]\npub fn tick(_ctx: &ReducerContext) {\n}",

API and ABI breaking changes

Expected complexity level and risk

1

Testing

  • Run the LLM benchmark and see if this fixes it

Copy link
Collaborator

@bfops bfops left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this helps, it seems good to me! Thank you!

@bfops bfops enabled auto-merge February 25, 2026 20:36
@bfops bfops added this pull request to the merge queue Feb 26, 2026
Merged via the queue into clockworklabs:master with commit 42097c3 Feb 26, 2026
27 checks passed
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.

2 participants