Skip to content

Update .gitignore and CI configuration#43

Merged
Looted merged 6 commits intomasterfrom
develop
Mar 9, 2026
Merged

Update .gitignore and CI configuration#43
Looted merged 6 commits intomasterfrom
develop

Conversation

@Looted
Copy link
Copy Markdown
Owner

@Looted Looted commented Mar 9, 2026

No description provided.

Copilot AI review requested due to automatic review settings March 9, 2026 09:34
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates packaging metadata and publishing workflow configuration, while cleaning up tracked per-branch KB artifacts that should not live in the repository.

Changes:

  • Adjusted bin paths / files formatting and updated repository.url in CLI and MCP package.json.
  • Removed committed .kb/branches/** artifacts and updated .gitignore rules for branch KB data.
  • Updated the npm publish GitHub Actions workflow to use Node.js 24 and added an explicit npm auth/tooling verification step.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
packages/mcp/package.json Tweaks package metadata for publishing (bin path, files list formatting, repository URL).
packages/cli/package.json Tweaks package metadata for publishing (bin path, files list formatting, repository URL).
.kb/branches/main/kb.rdf Removes previously tracked branch KB RDF file.
.kb/branches/develop/kb.rdf Removes previously tracked branch KB RDF file.
.kb/branches/develop/sync-cache.json Removes previously tracked sync cache artifact.
.gitignore Updates ignore patterns for .kb/branches and sync cache artifacts.
.github/workflows/publish.yml Updates Node version and adds a step to verify npm auth/tooling before publishing.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread packages/cli/package.json Outdated
"repository": {
"type": "git",
"url": "https://github.com/Looted/kibi.git"
"url": "git+https://github.com/Looted/kibi.git"
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

This switches repository.url to git+https://..., but other packages in the monorepo still use plain https://... (e.g. packages/core/package.json, packages/vscode/package.json). It’d be better to standardize the repository URL format across packages so registry metadata stays consistent.

Suggested change
"url": "git+https://github.com/Looted/kibi.git"
"url": "https://github.com/Looted/kibi.git"

Copilot uses AI. Check for mistakes.
Comment thread packages/mcp/package.json Outdated
"repository": {
"type": "git",
"url": "https://github.com/Looted/kibi.git"
"url": "git+https://github.com/Looted/kibi.git"
Copy link

Copilot AI Mar 9, 2026

Choose a reason for hiding this comment

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

This changes the repository URL format to git+https://..., but other packages in the repo still use plain https://... (e.g. packages/core/package.json, packages/vscode/package.json). Consider standardizing the repository.url format across packages to avoid inconsistent metadata on registries.

Suggested change
"url": "git+https://github.com/Looted/kibi.git"
"url": "https://github.com/Looted/kibi.git"

Copilot uses AI. Check for mistakes.
@Looted
Copy link
Copy Markdown
Owner Author

Looted commented Mar 9, 2026

@copilot open a new pull request to apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 9, 2026

@Looted I've opened a new pull request, #44, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 2 commits March 9, 2026 09:40
Co-authored-by: Looted <6255880+Looted@users.noreply.github.com>
Standardize repository.url format across all packages
@Looted Looted merged commit b614a69 into master Mar 9, 2026
1 check passed
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.

3 participants