[refactor] connect Open Library pages to Lark API by Copilot · Pull Request #100 · Open-Source-Bazaar/Open-Source-Bazaar.github.io

Copilot · 2026-07-23T10:22:33Z

This updates the shared search infrastructure to include Open Library books sourced from the Feishu-backed book dataset. Book search results now participate in the existing /search/[model] flow and render with the established Open Library card UI.

Search model integration
- Added SearchBookModel in models/Book.ts
- Registered book in SystemModel.searchMap so books can be queried through the common search entrypoint
Search result rendering
- Added the Open Library tab/label in pages/search/[model]/index.tsx
- Mapped book results to the existing Open Library book card presentation
API support for search
- Extended pages/api/open-library/books.ts to support:
  - keyword filtering
  - page-based slicing
  - total count response for the generic search page
- Preserved the original full-list response when no search/pagination params are provided, so existing callers keep working
Response shape
- Introduced a typed paginated response for book search results

export class SearchBookModel extends ListModel<Book, SearchBookFilter> {
  client = ownClient;
  baseURI = 'open-library/books';

  async loadPage(page = this.pageIndex, pageSize = this.pageSize, { keywords = '' }: SearchBookFilter = {}) {
    const { body } = await this.client.get<SearchBookPage>(
      `${this.baseURI}?${buildURLData({ keywords, page, pageSize })}`,
    );

    if (!body) throw new Error('Open Library book search API returned an empty response');

    return { pageData: body.data, totalCount: body.totalCount };
  }
}

Fixes 对接【开源图书馆】飞书多维表格 API #96

Human changes

[optimize] simplify AI codes of Open Library pages
[optimize] update Upstream packages

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

TechQuery

@copilot 在重复理解 issue 中的各条链接内容的前提下再写代码

Copilot

Copilot was unable to review this pull request because there is no eligible user to bill. To allow Copilot reviews on bot-authored pull requests, enable direct organization billing in your organization's Copilot settings.

Copilot · 2026-07-23T12:02:46Z

@copilot 在重复理解 issue 中的各条链接内容的前提下再写代码

已按 issue 里的链接重新对照后修改，现已改成走飞书表格数据源，并删除临时 books API。5934fe6

Copilot

Pull request overview

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

Comments suppressed due to low confidence (1)

pages/open-library/books/index.tsx:40

store={new BookModel()} creates a fresh store instance on every render, which can reset list state and trigger unnecessary network requests. Memoize the store instance so ScrollList can preserve pagination/loading state across renders.

        <ScrollList
          translator={i18n}
          store={new BookModel()}
          defaultData={books}

+export class BookModel extends BiDataTable<Book>() {
+  client = larkClient;
+
+  queryOptions: BiDataQueryOptions = { text_field_as_array: false };
+
+  constructor(appId = LibraryBaseId, tableId = BookTableId) {
+    super(appId, tableId);
+  }


+export class SearchBookModel extends BiSearch<Book>(BookModel) {
+  searchKeys = [
+    'title',
+    'isbn',
+    'authors',
+    'summary',
+    'recommendation',
+    'tags',
+    'donors',
+    'keepers',
+  ];
+}


Copilot

Pull request overview

Copilot reviewed 17 out of 19 changed files in this pull request and generated 2 comments.

Files not reviewed (1)

pnpm-lock.yaml: Generated file

Comments suppressed due to low confidence (2)

models/Book.ts:44

The PR description states that pages/api/open-library/books.ts was extended for keyword filtering/pagination and that search uses the /api/open-library/books endpoint, but this branch deletes that API route and implements book search via BiDataTable/BiSearch against Lark instead. Either the PR description needs updating, or the intended API-backed search flow is missing.

}

export class SearchBookModel extends BiSearch<Book>(BookModel) {
  searchKeys = [
    'title',

pages/open-library/book/[id].tsx:106

The ISBN label is hardcoded as ISBN while the surrounding metadata labels use t(...). This creates an untranslated user-facing string and breaks the project’s i18n pattern on this page.

                  <Col as="li">
                    <div className="text-muted small">ISBN</div>
                    <div>{isbn || t('not_available')}</div>
                  </Col>

 const SearchNameMap = ({ t }: typeof i18n): Record<string, string> => ({
  activity: t('activity'),
+  book: t('open_library'),
  project: t('open_source_projects'),
  NGO: t('NGO'),
 });


Initial plan

4bf610d

Copilot AI assigned Copilot and TechQuery Jul 23, 2026

Copilot AI review requested due to automatic review settings July 23, 2026 10:22

Copilot AI reviewed Jul 23, 2026

Copilot started work on behalf of TechQuery July 23, 2026 10:23 View session

Copilot AI linked an issue Jul 23, 2026 that may be closed by this pull request

对接【开源图书馆】飞书多维表格 API #96

Closed

TechQuery added the feature New feature or request label Jul 23, 2026

feat: add open library search model

069430c

Copilot AI review requested due to automatic review settings July 23, 2026 10:28

Copilot AI reviewed Jul 23, 2026

perf: cache open library search data

f8b9c87

Copilot AI review requested due to automatic review settings July 23, 2026 10:31

Copilot AI reviewed Jul 23, 2026

feat: paginate open library book search

1a0a128

Copilot AI review requested due to automatic review settings July 23, 2026 10:34

Copilot AI reviewed Jul 23, 2026

refactor: type open library search params

3a3face

Copilot AI review requested due to automatic review settings July 23, 2026 10:36

Copilot AI reviewed Jul 23, 2026

chore: polish open library search wiring

7370f60

Copilot AI review requested due to automatic review settings July 23, 2026 10:38

Copilot AI reviewed Jul 23, 2026

refactor: tidy open library books api

0418f0d

Copilot AI review requested due to automatic review settings July 23, 2026 10:41

Copilot AI reviewed Jul 23, 2026

refactor: harden open library search api

381634d

Copilot AI review requested due to automatic review settings July 23, 2026 10:43

Copilot AI reviewed Jul 23, 2026

chore: clarify open library search helpers

c5fb2f3

Copilot AI review requested due to automatic review settings July 23, 2026 10:45

Copilot AI reviewed Jul 23, 2026

Copilot AI changed the title ~~[WIP] Connect open library Feishu multi-dimensional table API~~ Add Open Library books to shared search flow Jul 23, 2026

Copilot finished work on behalf of TechQuery July 23, 2026 10:47

Copilot AI requested a review from TechQuery July 23, 2026 10:47

TechQuery requested changes Jul 23, 2026

View reviewed changes

Comment thread models/Book.ts Outdated

Comment thread pages/api/open-library/books.ts Outdated

Copilot started work on behalf of TechQuery July 23, 2026 11:39 View session

fix: switch open library search to lark data

5934fe6

Copilot AI review requested due to automatic review settings July 23, 2026 12:02

Copilot AI reviewed Jul 23, 2026

Copilot finished work on behalf of TechQuery July 23, 2026 12:04

Copilot AI requested a review from TechQuery July 23, 2026 12:04

[optimize] simplify AI codes of Open Library pages

555a5ee

Copilot AI review requested due to automatic review settings July 24, 2026 16:07

Copilot started reviewing on behalf of TechQuery July 24, 2026 16:08 View session

Copilot AI reviewed Jul 24, 2026

View reviewed changes

github-code-quality Bot found potential problems Jul 24, 2026

View reviewed changes

Comment thread models/Book.ts Fixed

[optimize] update Upstream packages

7ee05c6

Copilot AI review requested due to automatic review settings July 24, 2026 17:21

Copilot started reviewing on behalf of TechQuery July 24, 2026 17:22 View session

TechQuery marked this pull request as ready for review July 24, 2026 17:26

Copilot AI reviewed Jul 24, 2026

View reviewed changes

Comment thread pages/search/[model]/index.tsx

Comment on lines 44 to 49

const SearchNameMap = ({ t }: typeof i18n): Record<string, string> => ({

activity: t('activity'),

book: t('open_library'),

project: t('open_source_projects'),

NGO: t('NGO'),

});

Comment thread package.json

TechQuery approved these changes Jul 24, 2026

View reviewed changes

TechQuery changed the title ~~Add Open Library books to shared search flow~~ [refactor] connect Open Library pages to Lark API Jul 24, 2026

TechQuery merged commit 91c553c into main Jul 24, 2026
5 checks passed

TechQuery deleted the copilot/connect-open-library-feishu-api branch July 24, 2026 17:32

TechQuery mentioned this pull request Jul 29, 2026

Used AI to write the open-collaborator-award page #50

Open

3 tasks

Uh oh!

Conversation

Copilot AI commented Jul 23, 2026 • edited by TechQuery Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Human changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

TechQuery left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI commented Jul 23, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Jul 23, 2026 •

edited by TechQuery

Loading