Skip to content

Inheritables in root#249

Merged
janezd merged 4 commits into
biolab:masterfrom
janezd:inheritables-in-root
Jun 15, 2026
Merged

Inheritables in root#249
janezd merged 4 commits into
biolab:masterfrom
janezd:inheritables-in-root

Conversation

@janezd

@janezd janezd commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Fixes #242.

  • Parse inheritables in the root directory
  • Let ingest -p / ingest the root directory (non-recursively)
  • Improve SQL matches with LIKE in inheritables.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Enables parsing and ingestion of inheritable resources at the repository root, including support for ingest -p / to update only the root (non-recursively), and updates inheritable-resource lookups to use a generic query helper.

Changes:

  • Allow ingest -p / to target the root directory and route that mode to updateRoot.
  • Add inheritableResourcesFromPath and use it to ingest root-level inheritables.
  • Introduce a generic getInheritable DB helper and reuse it for CSS + favicon resolution.

Reviewed changes

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

Show a summary per file
File Description
scripts/ingest.ts Accept / as a special-case CLI path and document it in help text.
ingest/index.ts Skip per-prefix ingestion for / and ensure updateRoot runs when requested.
ingest/inheritables.ts Factor out root/prefix inheritable resource discovery into inheritableResourcesFromPath.
ingest/updatePaths.ts Reuse build-id creation, ingest root inheritables, and run cleanup for root.
api/content.ts Add a generic inheritable lookup and reuse it for CSS and favicon selection.

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

Comment thread ingest/updatePaths.ts
Comment thread ingest/updatePaths.ts

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread api/content.ts
Comment thread ingest/updatePaths.ts
Comment thread ingest/index.ts
@janezd janezd merged commit 6ca81a5 into biolab:master Jun 15, 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.

Inheritables in root directory

2 participants