Draft
Conversation
Updated all 11 alignment tracking files and both READMEs to plan Ruby language support for the temporal-developer skill. Determined which sections are applicable, Ruby-specific, or intentionally missing based on research of the Temporal Ruby SDK (GA, uses TracePoint + Durable Fiber Scheduler for determinism protection). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fixed critical alignment issues found during self-review: - workflow_run annotation doesn't exist; entry point is def execute - Ruby uses Cancellation tokens, not CancellationScope (TS concept) - ActiveModel mixin matches official SDK pattern - Fixed require paths and API call signatures Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Added Ruby sections to correctness tracking docs. Key findings: - patterns.md: External workflow signal syntax wrong (needs class method ref) - patterns.md: all_handlers_finished missing ? suffix - ruby.md: Client.connect missing namespace argument - determinism.md, determinism-protection.md, gotchas.md, observability.md: all good - Several files unchecked due to MCP rate limits (testing, versioning, advanced-features, data-handling, error-handling) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Fixed 3 verified issues: - External workflow signal syntax (class method ref) - all_handlers_finished? with Ruby boolean convention - Client.connect namespace argument Updated correctness tracking to mark all as FIXED. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Updated SKILL.md and core/determinism.md to include Ruby as a supported language alongside Python and TypeScript. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
donald-pinckney
changed the title
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.
Updates agent memory obtained from adding Ruby.