Conversation
| "use strict"; | ||
| var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) { | ||
| if (k2 === undefined) k2 = k; | ||
| var desc = Object.getOwnPropertyDescriptor(m, k); |
There was a problem hiding this comment.
probably shouldn't check in the dist folder
| throw new Error(`Invalid source-type: ${inputs.sourceType}. Must be one of: ${validSourceTypes.join(', ')}`); | ||
| } | ||
| // Validate source-specific inputs | ||
| switch (inputs.sourceType) { |
There was a problem hiding this comment.
this logic probably is on our side right? not in the git side? we intend them to use our agent system? which they would upload through us or through their agent repository? IE assume the agent is uploaded or do that in another step, (to simplify this) I'm not saying this has to be this way, more of a nit / idea but it will be a cleaner interface
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Deploy agent to Runloop | ||
| id: deploy |
There was a problem hiding this comment.
To my comment before there would be a step for upload agent
| "scripts": { | ||
| "build": "tsc", | ||
| "clean": "rm -rf dist", | ||
| "rebuild": "pnpm run clean && pnpm run build", |
There was a problem hiding this comment.
usually you don't do this, also probably recommended to use npm default if this is for showing other people what to do
|
|
||
| on: | ||
| release: | ||
| types: [published] |
There was a problem hiding this comment.
possibly merge main (this is true in a real environment but when someone copy pastas this they probably want main
| const fileName = path.basename(filePath); | ||
|
|
||
| // Determine content type based on file extension | ||
| let contentType: 'tgz' | 'tar' | 'gzip'; |
There was a problem hiding this comment.
Should probably delete this and just use the SDK code that does this.
| core.info(`Uploading single file: ${filePath}`); | ||
|
|
||
| const fileBuffer = fs.readFileSync(filePath); | ||
| const fileName = path.basename(filePath); |
| }, | ||
| ...(ttlMs && { ttl_ms: ttlMs }), | ||
| }; | ||
|
|
| const sample = buffer.slice(0, Math.min(1024, buffer.length)); | ||
| let textBytes = 0; | ||
|
|
||
| for (const byte of sample) { |
| "compilerOptions": { | ||
| "target": "ES2022", | ||
| "module": "commonjs", | ||
| "lib": ["ES2022"], |
There was a problem hiding this comment.
Ask the LLm to use Node configuration on this
2 participants
This has to use the old SDK for now since the new SDK does not (yet) support the agent APIs.