Skip to content

chore: update package name#9

Merged
JacobFV merged 1 commit intomainfrom
package_name_update
Feb 9, 2026
Merged

chore: update package name#9
JacobFV merged 1 commit intomainfrom
package_name_update

Conversation

@NamanGarg20
Copy link
Copy Markdown
Contributor

@NamanGarg20 NamanGarg20 commented Feb 9, 2026

Greptile Overview

Greptile Summary

Updates the npm package name from agi to @agi_inc/agi-js and refreshes the lockfile metadata (including optional platform binary versions).

Main merge blocker: package.json still contains a smoke test script that embeds an AGI_API_KEY literal; this should be removed so credentials are only supplied via environment/CI secrets.

Confidence Score: 3/5

  • This PR is mostly safe to merge but includes a credential leak that should be fixed first.
  • Changes are limited to package metadata, but a hardcoded API key in the test script is a real secret-management issue that should be removed before merging.
  • package.json

Important Files Changed

Filename Overview
package.json Renames the package to @agi_inc/agi-js; repo still contains a hardcoded AGI_API_KEY in the smoke test script that should be removed.
package-lock.json Updates lockfile package name and bumps optional platform binary packages to 0.4.0; no functional issues found in the lockfile changes.

Sequence Diagram

sequenceDiagram
  participant Dev as Developer/CI
  participant NPM as npm
  participant Pkg as package.json
  participant Test as vitest

  Dev->>NPM: npm run test:smoke
  NPM->>Pkg: read scripts.test:smoke
  Pkg-->>NPM: provide command string
  NPM->>Test: run smoke test
  Test-->>Dev: pass/fail
Loading

@NamanGarg20 NamanGarg20 changed the title chore: update package name and bump optional dependencies to 0.4.0 chore: update package name Feb 9, 2026
@NamanGarg20 NamanGarg20 requested a review from JacobFV February 9, 2026 23:22
Copy link
Copy Markdown
Contributor

@JacobFV JacobFV left a comment

Choose a reason for hiding this comment

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

lgtm

@JacobFV JacobFV merged commit 73466e0 into main Feb 9, 2026
12 checks passed
Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Feb 9, 2026

Additional Comments (1)

package.json
Hardcoded API key

test:smoke sets AGI_API_KEY to a literal value in the script, which will leak to anyone with read access and can be inadvertently used in CI. Please change the script to rely on a user/CI-provided env var (or omit setting it) and have the smoke test instruct/fail when AGI_API_KEY isn’t set.

Prompt To Fix With AI
This is a comment left during a code review.
Path: package.json
Line: 26:26

Comment:
**Hardcoded API key**

`test:smoke` sets `AGI_API_KEY` to a literal value in the script, which will leak to anyone with read access and can be inadvertently used in CI. Please change the script to rely on a user/CI-provided env var (or omit setting it) and have the smoke test instruct/fail when `AGI_API_KEY` isn’t set.

How can I resolve this? If you propose a fix, please make it concise.

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