-
Notifications
You must be signed in to change notification settings - Fork 126
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
dependency updates #590
dependency updates #590
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,17 @@ | ||
{ | ||
"cSpell.words": [ | ||
"genaiscript", | ||
"openai" | ||
], | ||
"genaiscript.cli.path": "../packages/cli/built/genaiscript.cjs" | ||
} | ||
"cSpell.words": ["genaiscript", "openai"], | ||
"genaiscript.cli.path": "../packages/cli/built/genaiscript.cjs", | ||
// https://hideoo.dev/notes/starlight-paste-images-with-visual-studio-code | ||
// Enable pasting files into a Markdown editor to create Markdown links. | ||
"markdown.editor.filePaste.enabled": "smart", | ||
// Copy pasted media files into a Markdown editor to the workspace. | ||
"markdown.editor.filePaste.copyIntoWorkspace": "mediaFiles", | ||
// Enable dropping files into a Markdown editor to create Markdown links. | ||
"markdown.editor.drop.enabled": "smart", | ||
// Copy dropped media files into a Markdown editor to the workspace. | ||
"markdown.editor.drop.copyIntoWorkspace": "mediaFiles", | ||
// Copy pasted files in documentation pages to the `src/assets/` directory. | ||
"markdown.copyFiles.destination": { | ||
"/src/content/docs/**/*": "/src/assets/" | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,25 @@ | ||
--- | ||
import { getCollection } from "astro:content" | ||
import { CardGrid, LinkCard } from "@astrojs/starlight/components" | ||
|
||
interface Props { | ||
directory: string | ||
} | ||
const { directory } = Astro.props | ||
|
||
const pages = ( | ||
await getCollection("docs", (entry) => entry.id.startsWith(directory + "/")) | ||
).filter(({ slug }) => slug !== directory) | ||
--- | ||
|
||
<CardGrid | ||
>{ | ||
pages.map(({ data, slug }) => ( | ||
<LinkCard | ||
title={data.title} | ||
href={"/" + slug} | ||
description={data.description} | ||
/> | ||
)) | ||
}</CardGrid | ||
> |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,6 +8,7 @@ keywords: GenAIScript CLI, Node.js automation, AI scripting, command line interf | |
|
||
import { Steps } from "@astrojs/starlight/components" | ||
import { Tabs, TabItem } from "@astrojs/starlight/components" | ||
import DirectoryLinks from '../../../../components/DirectoryLinks.astro'; | ||
|
||
The GenAIScript CLI **`genaiscript`** runs GenAIScript scripts | ||
outside of Visual Studio and in your [automation](/genaiscript/getting-starting/automating-scripts). | ||
|
@@ -136,3 +137,8 @@ npx genaiscript scripts model [script] | |
|
||
where [script] can be a script id or a file path. | ||
|
||
|
||
|
||
## Topics | ||
|
||
<DirectoryLinks directory="reference/cli" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The 'DirectoryLinks' component is used but not defined in the document.
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,8 +6,9 @@ description: Learn how to use and customize GenAIScript templates for efficient | |
keywords: script templates, AI prompts, prompt expansion, OpenAI integration, template customization | ||
--- | ||
import { FileTree } from "@astrojs/starlight/components" | ||
import DirectoryLinks from '../../../../components/DirectoryLinks.astro'; | ||
|
||
GenAIScript are JavaScript files named as `*.genai.mjs` | ||
GenAIScript are JavaScript files named as `*.genai.mjs`, or TypeScript files named as `*.genai.mts`, | ||
with a prompt creation engine designed by LLM prompting. | ||
|
||
```js title="shorten.genai.mjs" | ||
|
@@ -48,3 +49,8 @@ $`Shorten ${file}. Limit changes to minimum.` | |
|
||
- `system.*.genai.mjs` are considered [system prompt templates](/genaiscript/reference/scripts/system) | ||
and are unlisted by default. | ||
|
||
|
||
## Topics | ||
|
||
<DirectoryLinks directory="reference/scripts" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The 'DirectoryLinks' component is used but not defined in the document.
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,9 +8,6 @@ | |
"packages/*", | ||
"docs", | ||
"slides" | ||
], | ||
"nohoist": [ | ||
"json5" | ||
] | ||
}, | ||
"scripts": { | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -3,6 +3,7 @@ import { Command, program } from "commander" | |
export async function helpAll() { | ||
console.log(`---`) | ||
console.log(`title: Commands`) | ||
console.log(`description: List of all CLI commands`) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. You've added a new console.log statement to print a description. This is a good addition for clarity. However, it would be better to use template literals for consistency with the rest of the code. π
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The string you're logging doesn't follow the same format as the other console.log statements. It might be better to follow the same format for consistency. For example:
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The string "List of all CLI commands" is hard-coded. It would be better to store this in a variable or constant, especially if it's used in multiple places. This makes it easier to manage and update the text. π
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The added line
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The use of
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There is no error handling for the
|
||
console.log(`sidebar:`) | ||
console.log(` order: 100`) | ||
console.log(`---\n`) | ||
|
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.
There seems to be a typo here.