feat: Automate Agentica connector setup and validation & connector select test #983
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
lyutvs
changed the title
lyutvs
changed the title
8471919
reviewed
Apr 11, 2025
| @@ -1,4 +1,10 @@ | |||
| import { FileManager } from "@wrtnlabs/connector-shared/lib"; | |||
| ): string[] { | ||
| try { | ||
| if (!fs.existsSync(packagesPath)) { | ||
| console.warn(`⚠️ 경로를 찾을 수 없습니다: ${packagesPath}`); |
Contributor
There was a problem hiding this comment.
It would be nice if you could change these parts to English :)
Comment on lines
+84
to
+91
| await sendKey(" ", "스페이스 선택", 500); | ||
| await sendKey("\r", "엔터", 500); | ||
| } else { | ||
| console.warn(`⚠️ 유효하지 않은 connectorIndex: ${connectorIndex}`); | ||
| } | ||
|
|
||
| await sendKey("\x1B[D", "Yes ←"); | ||
| await sendKey("\r", "Yes 선택"); |
Contributor
There was a problem hiding this comment.
Currently, I'm now updating the agentica cli, so will be added environment variable input step between selecting connector step and entering openai apikey step.
Comment on lines
+192
to
+211
| // 3.1 ENV_LIST 찾기 (아직 못 찾은 경우) | ||
| if (!foundEnvList) { | ||
| const envListMatch = content.match( | ||
| /export\s+const\s+ENV_LIST\s*=\s*\[([\s\S]*?)\]/, | ||
| ); | ||
| if (envListMatch) { | ||
| const envListStr = envListMatch[1]; | ||
| const envVarMatches = envListStr.match(/'([^']+)'|"([^"]+)"/g); | ||
| if (envVarMatches) { | ||
| envList = envVarMatches.map((match) => | ||
| match.replace(/['"]/g, ""), | ||
| ); | ||
| console.log( | ||
| `[${connectorName}] Found ENV_LIST in ${filePath}:`, | ||
| envList, | ||
| ); | ||
| foundEnvList = true; // 찾았음을 표시 | ||
| } | ||
| } | ||
| } |
Contributor
There was a problem hiding this comment.
imo. in my opinion) In this part, I think you can use the dynamic import to get the const ENV_LIST.
Comment on lines
+324
to
+336
| if (hasFileManager) { | ||
| const awsEnvVars = [ | ||
| "AWS_ACCESS_KEY_ID", | ||
| "AWS_SECRET_ACCESS_KEY", | ||
| "AWS_S3_BUCKET", | ||
| "AWS_S3_REGION", | ||
| ]; | ||
| awsPropsStr = "{\n"; // 여기서 awsPropsStr 초기화 | ||
| awsEnvVars.forEach((envVar) => { | ||
| const camelCaseKey = toCamelCase(envVar); | ||
| awsPropsStr += ` ${camelCaseKey}: process.env.${envVar}!,\n`; | ||
| }); | ||
| awsPropsStr += " }"; // 들여쓰기 조정 |
Contributor
There was a problem hiding this comment.
In the future, new FileManager can be created like Azure, GCP, LocalFileSystem, and so on.
so this part will be changed someday.
Contributor
Author
|
I'll change agentica cli test why
|
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.
This pull request includes a series of updates across multiple files, focusing on dependency upgrades, refactoring, and the removal of unused modules. The most important changes are grouped by theme and listed below:
Dependency Upgrades:
package.jsonto their latest versions. This includes significant upgrades to packages such as@commitlint/cli,@nestia/sdk,typescript, and many others.Refactoring:
GoogleSlidesService,MarpService, andStableDiffusionBetaServiceclasses to remove direct dependency on theFileManagerclass, instead passing it through thepropsparameter. [1] [2] [3]IGoogleSlidesService,IMarpService, andIStableDiffusionBetaServicestructures to includefileManagerin theirIPropstypes. [1] [2] [3]Version Bump:
connectors-list.jsonfrom0.1.486to0.1.488.Removal of Unused Modules:
KeywordExtractController,KeywordExtractModule,LlmController, andLlmModulefiles, indicating that these modules are no longer in use. [1] [2] [3] [4]Minor Fixes:
pnpm-workspace.yamlby changing single quotes to double quotes for package exclusions.YoutubeOfficialSearchServicefor better readability. [1] [2]These changes collectively improve the codebase by updating dependencies, refactoring for better maintainability, and removing unused code.feat: Add script to automate Agentica connector setup and validation
This PR introduces a script to automate the process of setting up and performing initial validation for Agentica projects based on connectors found in
../../packages.Key Features:
packagesdirectory (respecting an ignore list).npx agentica startinteractively for each connector.ENV_LIST) and whether aFileManager(specificallyAwsS3Service) is needed..envfile in the created project (test-<connector-name>) with required variables (appending new ones, using placeholders).src/index.tsto inject environment variables andAwsS3Serviceinstance into the service constructor as needed.@wrtnlabs/connector-aws-s3ifFileManageris required.npm install,npm run build, andnpm run startfor basic validation.Benefits:
(Korean Summary / 한국어 요약)
../../packages내 커넥터들을 기반으로 Agentica 프로젝트 생성을 자동화하고 기본 검증을 수행하는 스크립트를 추가합니다. 각 커넥터에 대해agentica start실행, 필요 환경변수(ENV_LIST) 및FileManager사용 여부 분석,.env파일 생성/업데이트 (신규 변수 추가),src/index.ts수정 (환경변수 및AwsS3Service주입),npm install/build/start실행을 자동화하여 테스트 준비 과정을 단축하고 일관성을 확보합니다.