Skip to content

add repository URL to package.json files for npm #1370

Merged
joshunrau merged 2 commits into
DouglasNeuroInformatics:mainfrom
joshunrau:release-changes
Jun 3, 2026
Merged

add repository URL to package.json files for npm #1370
joshunrau merged 2 commits into
DouglasNeuroInformatics:mainfrom
joshunrau:release-changes

Conversation

@joshunrau

@joshunrau joshunrau commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Updated package repository metadata for improved project documentation.
    • Refined internal implementation details across API and admin services.

@joshunrau joshunrau merged commit 32081f4 into DouglasNeuroInformatics:main Jun 3, 2026
1 check was pending
@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: e58bdc9a-888b-41e9-bfe5-0304a2794982

📥 Commits

Reviewing files that changed from the base of the PR and between 91bc110 and 6c08117.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (5)
  • apps/api/src/demo/demo.service.ts
  • apps/web/src/routes/_app/admin/users/index.tsx
  • packages/instrument-bundler/package.json
  • packages/instrument-guidelines/package.json
  • packages/serve-instrument/package.json

Walkthrough

The PR contains minor code adjustments and package metadata updates. The demo service removes an explicit type cast when passing questionnaire data, the admin users form reorders a field assignment, and three workspace packages receive repository metadata blocks.

Changes

Code and metadata maintenance

Layer / File(s) Summary
Code adjustments in service and form
apps/api/src/demo/demo.service.ts, apps/web/src/routes/_app/admin/users/index.tsx
Demo service instrument-record creation removes the explicit as Json cast on the questionnaire data. Admin users form reorders the selectedUserBasePermission field assignment to after the initialValues conditional block.
Package repository metadata
packages/instrument-bundler/package.json, packages/instrument-guidelines/package.json, packages/serve-instrument/package.json
Three workspace packages now include repository metadata blocks specifying git type, repository URL, and package directory paths.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Possibly related PRs

  • DouglasNeuroInformatics/OpenDataCapture#1004: The change to DemoService.init removes the as Json cast on questionnaire data passed to instrument-record creation, which connects to that PR's updates to how InstrumentRecordsService handles incoming record data.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install failed: dependency version conflict. Check your lock file or package.json.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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