Skip to content

Add datetime node builder - #7

Merged
andres-javier-lopez merged 3 commits into
mainfrom
feat/add-datetime-node-builder
Oct 3, 2025
Merged

andres-javier-lopez merged 3 commits into
mainfrom
feat/add-datetime-node-builder

Conversation

@andres-javier-lopez

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new datetime node builder that provides current date and time information to prompts. The implementation includes helper functions for getting weekday and current time, a DateTimeNode class that extends PromptNode, and a DateTimeBuilder class for integrating the datetime functionality into the assistant core.

Key changes:

  • Creates a new datetime builder module with timezone-aware date/time functionality
  • Implements comprehensive unit tests with deterministic datetime mocking
  • Updates the changelog to document the new feature

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
assistant_core/builder/datetime.py Implements DateTimeNode and DateTimeBuilder classes with timezone support
tests/unit/test_datetime_builder.py Comprehensive test suite with mocking for deterministic datetime testing
CHANGELOG.md Documents the new datetime builder feature

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment thread assistant_core/builder/datetime.py Resolved
Comment thread assistant_core/builder/datetime.py Resolved
@andres-javier-lopez
andres-javier-lopez merged commit 93208a0 into main Oct 3, 2025
1 check passed
@andres-javier-lopez
andres-javier-lopez deleted the feat/add-datetime-node-builder branch October 3, 2025 22:32
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