Skip to content

fix: Added missing package.json fields: license, bugs, repository - #123

Open
Omission-create wants to merge 1 commit into
DevilsAutumn:mainfrom
Omission-create:auto-fix-metadata-1319
Open

fix: Added missing package.json fields: license, bugs, repository#123
Omission-create wants to merge 1 commit into
DevilsAutumn:mainfrom
Omission-create:auto-fix-metadata-1319

Conversation

@Omission-create

Copy link
Copy Markdown

Auto-fix

Added missing package.json fields: license, bugs, repository

Related: #1319

@greptile-apps

greptile-apps Bot commented Jun 12, 2026

Copy link
Copy Markdown

Greptile Summary

This PR adds three missing package.json metadata fields — license, bugs, and repository — intended to satisfy standard npm package conventions. However, two of the three added fields contain incorrect values that need to be corrected before merging.

  • license: "MIT" is added correctly.
  • bugs.url is set to the template placeholder https://github.com/owner/repo/issues rather than the actual repository URL (https://github.com/devilsautumn/quater/issues).
  • repository.url is an empty string, making the field non-functional; it should be git+https://github.com/devilsautumn/quater.git.

Confidence Score: 3/5

The PR introduces incorrect metadata: the bugs URL is a literal placeholder and the repository URL is empty — both added fields are non-functional as written.

Two of the three new fields contain wrong values. The bugs.url still points to owner/repo (an unfilled template), and repository.url is an empty string. Neither value will work correctly with npm tooling or GitHub integrations that read these fields.

package.json — the bugs.url and repository.url values need to be updated to the actual repository coordinates before this is merged.

Important Files Changed

Filename Overview
package.json Adds license, bugs, and repository fields, but bugs.url uses an unfilled placeholder (owner/repo) and repository.url is an empty string — both values are incorrect and need to be set to the actual repository URL.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[package.json metadata fields] --> B[license: MIT]
    A --> C[bugs.url]
    A --> D[repository.url]
    C --> E[Placeholder owner/repo - should be devilsautumn/quater]
    D --> F[Empty string - should be git+https://github.com/devilsautumn/quater.git]
Loading

Fix All in Codex Fix All in Claude Code

Reviews (1): Last reviewed commit: "fix: Added missing package.json fields: ..." | Re-trigger Greptile

Comment thread package.json
Comment on lines +24 to +26
"bugs": {
"url": "https://github.com/owner/repo/issues"
},

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 The bugs.url still uses the template placeholder owner/repo instead of the actual repository path (devilsautumn/quater). This means bug reports will be directed to a non-existent GitHub URL.

Suggested change
"bugs": {
"url": "https://github.com/owner/repo/issues"
},
"bugs": {
"url": "https://github.com/devilsautumn/quater/issues"
},

Fix in Codex Fix in Claude Code

Comment thread package.json
Comment on lines +27 to 30
"repository": {
"type": "git",
"url": ""
}

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 The repository.url is an empty string, which makes the field effectively useless. npm expects a valid git URL here (typically in git+https:// or git:// format). An empty value can cause tooling that reads this field to fail or produce incorrect output.

Suggested change
"repository": {
"type": "git",
"url": ""
}
"repository": {
"type": "git",
"url": "git+https://github.com/devilsautumn/quater.git"
}

Fix in Codex Fix in Claude Code

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