DOP-5036 Refactor Search-Manifest integration #10
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,31 @@ | ||
| { | ||
| "$schema": "https://biomejs.dev/schemas/1.9.1/schema.json", | ||
| "vcs": { | ||
| "enabled": false, | ||
| "clientKind": "git", | ||
| "useIgnoreFile": true | ||
| }, | ||
| "files": { | ||
| "ignoreUnknown": false, | ||
| "ignore": ["*/.ntli/*"] | ||
| }, | ||
| "formatter": { | ||
| "enabled": true, | ||
| "indentStyle": "tab" | ||
| }, | ||
| "organizeImports": { | ||
| "enabled": true | ||
| }, | ||
| "linter": { | ||
| "enabled": true, | ||
| "rules": { | ||
| "recommended": true | ||
| } | ||
| }, | ||
| "javascript": { | ||
| "formatter": { | ||
| "quoteStyle": "single", | ||
| "trailingCommas": "all" | ||
| } | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,24 @@ | ||
| import type { envVars } from "./types"; | ||
|
|
||
| const assertEnvVars = (vars: envVars) => { | ||
| const missingVars = Object.entries(vars) | ||
| .filter(([, value]) => !value) | ||
| .map(([key]) => `- ${key}`) | ||
| .join("\n"); | ||
| if (missingVars) | ||
| throw new Error(`Missing env var(s) ${JSON.stringify(missingVars)}`); | ||
| return vars; | ||
| }; | ||
|
|
||
| export const getEnvVars = () => { | ||
| const environmentVariables = assertEnvVars({ | ||
| ATLAS_CLUSTER0_URI: `mongodb+srv://${process.env.MONGO_ATLAS_USERNAME}:${process.env.MONGO_ATLAS_PASSWORD}@${process.env.MONGO_ATLAS_CLUSTER0_HOST}/?retryWrites=true&w=majority`, | ||
| SNOOTY_DB_NAME: `${process.env.MONGO_ATLAS_POOL_DB_NAME}`, | ||
| ATLAS_SEARCH_URI: `mongodb+srv://${process.env.MONGO_ATLAS_USERNAME}:${process.env.MONGO_ATLAS_PASSWORD}@${process.env.MONGO_ATLAS_SEARCH_HOST}/?retryWrites=true&w=majority`, | ||
| SEARCH_DB_NAME: `${process.env.MONGO_ATLAS_SEARCH_DB_NAME}`, | ||
| REPOS_BRANCHES_COLLECTION: "repos_branches", | ||
| DOCSETS_COLLECTION: "docsets", | ||
| DOCUMENTS_COLLECTION: "documents", | ||
| }); | ||
| return environmentVariables; | ||
| }; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,25 +1,20 @@ | ||
| import { NetlifyIntegration } from "@netlify/sdk"; | ||
|
|
||
| export class Facet { | ||
| category: string; | ||
| value: string; | ||
| subFacets: Array<Facet>; | ||
|
|
||
| constructor(category: string, value: string, subFacets: Array<Facet>) { | ||
| this.category = category; | ||
| this.value = value; | ||
| this.subFacets = subFacets; | ||
| if (subFacets) { | ||
| for (const subFacet of subFacets) { | ||
|
There was a problem hiding this comment. We don't need this if statement as There was a problem hiding this comment. actually, |
||
| this.subFacets.push( | ||
| new Facet(subFacet.category, subFacet.value, subFacet.subFacets ?? []) | ||
| ); | ||
|
There was a problem hiding this comment. If the There was a problem hiding this comment. Thanks for catching this as well- There was a problem hiding this comment. Gotcha! I was also wondering if it makes sense to create a new interface Facet {
category: string;
value: string;
subFacet: Array<Facet>;
} |
||
| } | ||
| } | ||
| } | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need a
biome.jsonfor this directory as it finds the nearestbiome.jsonin the project structure.I think it would be preferable to use the single top level one so that the configurations are consistent, but I'm not opposed to adding one here if there's a specific need.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No specific need!