-
Notifications
You must be signed in to change notification settings - Fork 5.5k
Templates restructure #7816
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
Merged
Parent:
AI Components PR Review
andrewjschuang
merged 8 commits into
ai-components/pr-review
from
ai-components/templates-restructure
Aug 30, 2023
Merged
Templates restructure #7816
andrewjschuang
merged 8 commits into
ai-components/pr-review
from
ai-components/templates-restructure
Aug 30, 2023
Conversation
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
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dylburger
reviewed
Aug 29, 2023
packages/component_code_gen/code_gen/generate_component_code.py
Outdated
Show resolved
Hide resolved
dylburger
reviewed
Aug 30, 2023
packages/component_code_gen/templates/actions/additional_rules.py
Outdated
Show resolved
Hide resolved
dylburger
approved these changes
Aug 30, 2023
andrewjschuang
added a commit
that referenced
this pull request
Aug 30, 2023
* add generate webhook sample * add generate webhook source * modify logic for main * add tests * add component metadata rules (sources) * add component metadata rules (actions) * make only one request to create actions * make gpt model a config * try and go to next if failed * add component metadata to examples * modify source key/name to past tense * remove generate webhook sample * add webhook signature validation * add webhook activate and deactivate hooks * add brex test webhook source * redirect all test output to file * add deploy hook * fix typo * add env example * change order in readme * change main function name * add code example with auth to templates * change instructions for running * disable docs by default * add tests to readme * add tests for actions * read instructions file outside of main * change tests to python * return copy of template instead of overwriting * use only one script for running tests * change component_type arg to type * add $summary export * add azure openai option and adjust configs * add polling source template * add tests for polling components * alphabetical reorder tests * add apps template * Testing Stable Diffusion image to text * Adding pnpm.lock * pnpm.lock * Multi agent debate (#7787) * Support for multi-agent debate * pnpm.lock * Improving readme, updating tool-versions * Update generate_component_code.py * Templates restructure (#7816) * requested changes * templates organization * add auth authentication example * modify config default values * add newlines * fix case when there is no code scaffolding * separate common rules and additional rules --------- Co-authored-by: Dylan J. Sather <dylan.sather@gmail.com> Co-authored-by: Danny Roosevelt <danny@pipedream.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
WHAT
🤖 Generated by Copilot at 2954825
This pull request improves the code generation logic and the quality of the generated code for Pipedream components. It refactors and simplifies some configuration and helper files, and adds new templates and variables for the language model prompts and system instructions. It also fixes a typo in the README file and passes the authentication code example to the language model.
🤖 Generated by Copilot at 2954825
WHY
HOW
🤖 Generated by Copilot at 2954825
generate_codefunction to improve readability and performance (link, link, link, link, link)get_env_varfunction andconfigobject to simplify configuration and avoid hardcoding (link, link)auth_exampleparameter toPipedreamOpenAPIAgentclass andcall_langchainfunction to pass authentication code example to language model (link, link, link)templates/actionsmodule to provide text for introduction, examples, and additional rules for generating action components (link, link, link, link, link)templates/appsmodule to provide text for introduction, examples, auth, methods, and prop definitions for generating app code (link, link, link, link, link, link)templates/commonmodule to provide text for common explanations and instructions for generating component code (link, link, link, link, link)