Skip to content

Update package.json#9

Open
scottsgeorge wants to merge 1 commit into
mainfrom
scottsgeorge-patch-15
Open

Update package.json#9
scottsgeorge wants to merge 1 commit into
mainfrom
scottsgeorge-patch-15

Conversation

@scottsgeorge
Copy link
Copy Markdown
Contributor

Paywaz Pull Request

Thank you for contributing to Paywaz!
Please complete the sections below to help us review your pull request efficiently.


Summary

Describe the purpose of this PR.
Example: “Fixes SDK authentication bug”, “Adds Token Conversion API support”, “Updates TypeScript types”.


Issue Reference (Optional)

Link related issues, tickets, or discussions:

  • Closes #
  • Related to #

Changes Introduced

List the key changes:


Type of Change

Select all that apply:

  • Bug fix
  • New feature
  • Code cleanup / refactor
  • Documentation update
  • Security fix
  • Breaking change (requires major version bump)

If breaking change, explain impact:


How to Test

Explain how reviewers can test this PR:

Include commands if relevant:

npm install
npm run test
npm run build


Screenshots / Logs (Optional)

Attach images or logs if helpful.


Documentation

Does this require documentation updates?

  • Yes
  • No

If yes, describe the changes needed.


Security Considerations

Does this change affect authentication, API calls, transactions, or sensitive logic?

  • No
  • Yes — Please explain:

Checklist

Before submitting:

  • Code follows project style guidelines
  • Linting passes (npm run lint)
  • Tests added or updated
  • All tests pass locally
  • Documentation updated (if needed)
  • I have reviewed the Paywaz License terms

Thank you for improving the Paywaz ecosystem!
Your contribution helps accelerate zero-fee global payments.

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread package.json
Comment on lines 1 to +5
{
"name": "@paywaz/sdk",
"version": "0.1.0",
"description": "Paywaz JavaScript SDK",
"main": "dist/index.js",
"types": "dist/index.d.ts",
"exports": {
".": {
"import": "./dist/index.js",
"types": "./dist/index.d.ts"
}
},
"files": [
"dist"
],
"scripts": {
"build": "tsc",
"generate:sdk": "ts-node scripts/generate-sdk.ts",
"docs": "typedoc",
"prepublishOnly": "npm run build"
},
"dependencies": {},
"devDependencies": {
"typescript": "^5.0.0",
"ts-node": "^10.9.0",
"typedoc": "^0.25.0",
"@openapitools/openapi-generator-cli": "^2.7.0"
}
}
{
"devDependencies": {
"@changesets/cli": "^2.27.0",
"typescript": "^5.0.0",
"ts-node": "^10.9.0",
"typedoc": "^0.25.0",
"@openapitools/openapi-generator-cli": "^2.7.0"
},
"scripts": {
"build": "tsc",
"generate:sdk": "ts-node scripts/generate-sdk.ts",
"docs": "typedoc",
"prepublishOnly": "npm run build",
"extends": "./tsconfig.base.json",
"compilerOptions": {
"rootDir": "src",
"outDir": "dist",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge package.json replaced with tsconfig options

The commit rewrites package.json to contain TypeScript compilerOptions instead of a package manifest, removing required fields like name/version and all scripts/dependencies. With this file, npm will not treat the repo as a package (npm requires name and version) and existing workflows that run npm install, npm run build, or publish to npm will fail because the manifest and scripts no longer exist.

Useful? React with 👍 / 👎.

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.

1 participant