-
Notifications
You must be signed in to change notification settings - Fork 0
Genie 1171/story/create template system #67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
odaiodeh
wants to merge
32
commits into
main
Choose a base branch
from
GENIE-1171/STORY/Create-template-system
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 17 commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
158df93
Draft | First imp | Agentic AI Templates
Nirsisr 1d25518
ADD template service
odaiodeh ca44c04
FIX json schema requiered and fix the ref in plan to not be included โฆ
odaiodeh 5e6e80d
Templates | Followed BE APIs calls | Stage 2
Nirsisr a774730
FieldValidation | Inject dep
Nirsisr e2d4838
ADD bearer token to mcp provider
odaiodeh 4fc9fbd
Merge remote-tracking branch 'origin/GENIE-1167/EPIC/Agentic-AI-Templโฆ
odaiodeh f14ea61
Refactor the template code to be cleaner and SOLID, ADD README
odaiodeh 4e467c8
Apply suggestion from @coderabbitai[bot]
Nirsisr 657e9b2
CodeRaabit | Config mistake | Typo
Nirsisr 0121bbe
ADD template id in user session chat metadata
odaiodeh 555ad42
Merge remote-tracking branch 'origin/GENIE-1167/EPIC/Agentic-AI-Templโฆ
odaiodeh 696c9cc
remove comments out
odaiodeh 6c3b039
- Sidebar | Agentic AI | Change Order
Nirsisr 299fa3a
Apply suggestion from @coderabbitai[bot]
Nirsisr bad4e45
Merge remote-tracking branch 'origin/GENIE-1167/EPIC/Agentic-AI-Templโฆ
odaiodeh 255fd0b
Error Logic | Check by `severity`
Nirsisr 504c66a
FIX return of blueprint validation error list
odaiodeh 50c62aa
FIX missing usage of tags in list templates api endpoint
odaiodeh 436c496
FIX Materialize should preserve instantiation error contract and resoโฆ
odaiodeh 4b9fd2a
ADD TODO for authorization checks on all delete endpoints (templates,โฆ
odaiodeh c129e08
- Simplify delete_template endpoint to use TemplateNotFoundError
odaiodeh 83874ff
Apply suggestion from @coderabbitai[bot]
Nirsisr 6f34afd
CodeRaabit | Suggestion fixes
Nirsisr 8435a51
Merge branch 'GENIE-1167/EPIC/Agentic-AI-Templates' of github.com:redโฆ
Nirsisr f853468
FIX the max worker to be min 1
odaiodeh 5097673
Merge remote-tracking branch 'origin/GENIE-1167/EPIC/Agentic-AI-Templโฆ
odaiodeh 1cf0227
CHANGE init of mcp provider to be by factory in validator and actions
odaiodeh 7d1d32e
FieldValidation | Re-trigger Validaiton Action when dependency being โฆ
Nirsisr fdbeeb0
Merge remote-tracking branch 'origin/GENIE-1167/EPIC/Agentic-AI-Templโฆ
odaiodeh 97b42fa
Template User Fill | Flip onClick logic
Nirsisr f060a2e
Merge remote-tracking branch 'origin/GENIE-1167/EPIC/Agentic-AI-Templโฆ
odaiodeh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mask bearer_token in the action schema (SecretHint).
The token is sensitive; without SecretHint the UI may render plaintext and risk exposure. Align this with
McpProviderConfigby adding SecretHint hints.๐ Proposed fix
class ValidateConnectionInput(BaseActionInput): """Input for MCP connection validation""" sse_endpoint: HttpUrl bearer_token: Optional[str] = Field( default=None, - description="Bearer token for MCP server authentication" + description="Bearer token for MCP server authentication", + json_schema_extra=SecretHint(reason="API credentials should be masked").to_hints() )๐ค Prompt for AI Agents