Skip to content

Add bbop-response-golr package#24

Merged
pkalita-lbl merged 1 commit into
mainfrom
issue-12-bbop-response-golr
May 4, 2026
Merged

Add bbop-response-golr package#24
pkalita-lbl merged 1 commit into
mainfrom
issue-12-bbop-response-golr

Conversation

@pkalita-lbl

Copy link
Copy Markdown
Member

Fixes #12

These changes add the source code for bbop-response-golr as a new workspace packag

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds the legacy bbop-response-golr response handler into the monorepo as a new workspace package, with Node.js LTS-compatible packaging and a basic unit test suite.

Changes:

  • Introduces bbop-response-golr source (src/response.js) implementing GOlr/Solr response helpers (paging, facets, filters, doc lookup, highlighting).
  • Adds unit tests for core response behaviors (src/response.test.js).
  • Registers the new workspace in package-lock.json and adds package metadata (package.json, README.md).

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
packages/bbop-response-golr/src/response.js New GOlr response parsing/helper implementation.
packages/bbop-response-golr/src/response.test.js New tests covering success/paging/facets/filters and doc helper methods.
packages/bbop-response-golr/README.md New package README describing purpose and availability.
packages/bbop-response-golr/package.json New workspace package definition (exports/build/test/deps).
package-lock.json Adds the new workspace package to the lockfile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/bbop-response-golr/src/response.js
Comment thread packages/bbop-response-golr/src/response.js
Comment thread packages/bbop-response-golr/src/response.js
Comment thread packages/bbop-response-golr/src/response.js
@pkalita-lbl
pkalita-lbl merged commit 011cbd3 into main May 4, 2026
5 checks passed
@pkalita-lbl
pkalita-lbl deleted the issue-12-bbop-response-golr branch May 4, 2026 22:14
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.

Move bbop-response-golr source into monorepo

2 participants