Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check MDX frontmatter #96

Merged
merged 18 commits into from
Sep 24, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
109 changes: 109 additions & 0 deletions .github/scripts/check-mdx-frontmatter.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,109 @@
const fs = require("fs").promises;
const path = require("path");
const matter = require("gray-matter");

const mdxDir = path.join(__dirname, "../../fern/pages");
const filePattern = /\.mdx$/;

// Counters
let totalFilesChecked = 0;
let totalFilesValid = 0;
let totalFilesInvalid = 0;

// List of folders to exclude (relative to mdxDir)
const excludedFolders = ["-ARCHIVE-", "api-reference", "llm-university"];

function shouldExcludeFolder(dirPath) {
return excludedFolders.some((excludedFolder) => {
return path.relative(mdxDir, dirPath).startsWith(excludedFolder);
});
}

async function shouldExcludeFile(filePath) {
try {
const fileContent = await fs.readFile(filePath, "utf8");
const { data } = matter(fileContent);
return data.hidden === true;
} catch (error) {
console.error(`Error reading file "${filePath}":`, error);
return false; // In case of error, don't exclude the file
}
}

async function checkDescriptionLength(filePath) {
totalFilesChecked++;
const fileContent = await fs.readFile(filePath, "utf8");
const { data } = matter(fileContent);

if (!data.description) {
console.log(`File "${filePath}" is missing a description.`);
totalFilesInvalid++;
return false;
}

const descriptionLength = data.description.length;

if (descriptionLength < 50 || descriptionLength > 160) {
console.log(
`File "${filePath}" has an invalid description length: ${descriptionLength} characters.`
);
totalFilesInvalid++;
return false;
}

totalFilesValid++;
return true;
}

async function checkMDXFiles(dirPath) {
let allFilesValid = true;
const files = await fs.readdir(dirPath);

for (const file of files) {
const fullPath = path.join(dirPath, file);
const stat = await fs.lstat(fullPath);

if (stat.isDirectory()) {
if (shouldExcludeFolder(fullPath)) {
console.log(`Skipping excluded directory: ${fullPath}`);
continue;
}
const isValid = await checkMDXFiles(fullPath);
if (!isValid) {
allFilesValid = false;
}
} else if (filePattern.test(file)) {
if (await shouldExcludeFile(fullPath)) {
console.log(`Skipping excluded file: ${fullPath}`);
continue;
}
const isValid = await checkDescriptionLength(fullPath);
if (!isValid) {
allFilesValid = false;
}
}
}

return allFilesValid;
}

(async () => {
const allFilesValid = await checkMDXFiles(mdxDir);

// Summary report
console.log(`\nSummary Report:`);
console.log(`Total files checked: ${totalFilesChecked}`);
console.log(`Total valid files: ${totalFilesValid}`);
console.log(`Total invalid files: ${totalFilesInvalid}`);

if (!allFilesValid) {
console.error(
"Some files have invalid or missing descriptions. Meta description needing to be 50-160 characters"
);
process.exit(1); // Fail if any file is invalid
} else {
console.log(
"All files have a valid description length in the frontmatter."
);
}
})();
29 changes: 29 additions & 0 deletions .github/workflows/check-mdx-frontmatter.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
name: check-mdx-frontmatter

on:
pull_request:
branches:
- main
paths:
- 'fern/pages/**/*.mdx'
- 'fern/pages/**/**/*.mdx'

jobs:
run:
runs-on: ubuntu-latest
permissions: write-all
steps:
- name: Checkout repository
uses: actions/checkout@v4

- name: Setup pnpm
uses: pnpm/action-setup@v2
with:
version: 8

- name: Install Dependencies
shell: bash
run: pnpm install

- name: Run MDX frontmatter check
run: node .github/scripts/check-mdx-frontmatter.js
2 changes: 1 addition & 1 deletion fern/pages/responsible-use/security.mdx
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,4 @@ createdAt: "Wed Sep 14 2022 22:54:55 GMT+0000 (Coordinated Universal Time)"
updatedAt: "Mon Oct 23 2023 14:40:59 GMT+0000 (Coordinated Universal Time)"
type: "link"
link_url: "https://cohere.ai/security"
---
---
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
"tailwindcss": "^3.4.4"
},
"dependencies": {
"gray-matter": "^4.0.3",
"fern-api": "^0.41.16",
"react": "^18.3.1"
}
Expand Down
78 changes: 78 additions & 0 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading