@shawnstack/quickforge 1.3.27 → 1.3.29

package/plugins/openai-spreadsheets/plugin.json

@@ -0,0 +1,20 @@
+{
+  "name": "openai-spreadsheets",
+  "displayName": "OpenAI Spreadsheets",
+  "version": "0.1.0",
+  "description": "Spreadsheet planning, analysis, modeling, table design, chart specification, and workbook review workflows for QuickForge agents.",
+  "apiVersion": 1,
+  "quickforgeVersion": ">=1.3.25",
+  "enabledByDefault": true,
+  "contributes": {
+    "skills": "skills",
+    "commands": "commands"
+  },
+  "permissions": {
+    "filesystem": ["read", "workspace-write"],
+    "shell": "approval-required",
+    "network": "disabled",
+    "secrets": "no-secrets",
+    "git": ["read"]
+  }
+}

package/plugins/openai-spreadsheets/skills/spreadsheets/SKILL.md

@@ -0,0 +1,124 @@
+---
+name: spreadsheets
+description: Use this skill when the user asks to create, edit, analyze, model, review, visualize, or structure spreadsheet workbooks such as XLSX-oriented files, CSV/TSV tables, trackers, dashboards, financial models, formulas, charts, or data-analysis sheets.
+metadata:
+  displayName: Spreadsheets
+  version: 0.1.0
+  tags:
+    - spreadsheets
+    - xlsx
+    - csv
+    - dashboards
+    - analysis
+    - formulas
+---
+
+# Spreadsheets skill
+
+Use this skill to create or improve spreadsheet-oriented deliverables. A good spreadsheet is organized, auditable, readable, and fit for the user's purpose. For analysis-heavy tasks, correctness matters as much as formatting.
+
+This QuickForge adaptation intentionally does **not** depend on Codex cache paths, Codex workspace dependencies, or `@oai/artifact-tool`. Prefer spreadsheet plans, CSV/TSV outputs, formulas, workbook specs, or Markdown tables first. If the user explicitly needs XLSX generation or editing, explain the available local workflow and ask before using shell commands or external tools.
+
+## When to use
+
+Use this skill for requests involving:
+
+- Creating trackers, schedules, budgets, calculators, models, dashboards, reports, or tabular templates.
+- Working with `.xlsx`, `.xls`, `.csv`, `.tsv`, or Google Sheets-targeted spreadsheet content.
+- Designing formulas, validations, tables, pivots, charts, or workbook structure.
+- Analyzing spreadsheet data or answering questions about provided tabular files.
+- Reviewing or improving workbook layout, formulas, assumptions, or data quality.
+
+## Spreadsheet principles
+
+1. **Separate purpose, inputs, calculations, and outputs.** For non-trivial workbooks, avoid mixing everything on one sheet.
+2. **Make assumptions visible.** Put assumptions near the top or in a dedicated sheet/section.
+3. **Use clear labels and units.** Headers should be unambiguous.
+4. **Prefer auditable formulas.** Explain formulas and avoid hidden magic.
+5. **Design for maintenance.** Tables should be easy to extend.
+6. **Validate important outputs.** Include checks for reconciliations, totals, or formula consistency when relevant.
+7. **Use charts only when they add insight.** Every chart should have a takeaway.
+
+## Recommended workbook structures
+
+### Simple tracker/template
+
+- Title and usage note.
+- Main table with clear columns.
+- Status or category values if useful.
+- Summary section only if it helps.
+
+### Analysis workbook
+
+- Executive summary or dashboard.
+- Source data.
+- Assumptions.
+- Calculations/model.
+- Outputs/charts.
+- Checks, if correctness depends on formulas or reconciliations.
+
+### Financial or operations model
+
+- Summary/dashboard.
+- Inputs/assumptions.
+- Historical/source data.
+- Model/calculations.
+- Scenarios/sensitivity.
+- Checks.
+
+## Recommended workflow
+
+### New spreadsheet-oriented deliverable
+
+1. Identify the user goal, output format, data sources, and level of rigor.
+2. Choose workbook/sheet structure before writing formulas or tables.
+3. Draft the table/schema/formulas/charts.
+4. Validate calculations and edge cases.
+5. If writing to disk, use CSV/TSV/Markdown when sufficient; ask before attempting XLSX generation.
+
+### Editing or analyzing existing spreadsheet data
+
+1. Read the relevant file(s) first.
+2. Determine whether the user wants an answer, a file edit, or a new artifact.
+3. Preserve existing formulas and patterns unless asked to refactor.
+4. For formula changes, explain changed logic.
+5. For data analysis, call out data quality risks and assumptions.
+
+## Formula guidance
+
+- Prefer formulas that are understandable to spreadsheet users.
+- Use consistent row/column references.
+- Document non-obvious formulas.
+- Avoid volatile functions unless needed.
+- For financial models, keep assumptions editable and outputs linked.
+
+## Chart guidance
+
+When specifying charts, include:
+
+- Chart type.
+- Source data range or fields.
+- X/Y axes.
+- Series.
+- Main takeaway.
+- Any annotations or formatting guidance.
+
+Do not create charts with invented data. If data is missing, ask for it or propose placeholders.
+
+## XLSX / Google Sheets-oriented requests
+
+QuickForge currently provides this as a spreadsheet-planning/workflow skill, not a bundled spreadsheet runtime. For XLSX or Google Sheets deliverables:
+
+- First create a precise workbook spec, CSV/TSV data, or formulas.
+- If XLSX creation/editing is required, ask before running shell commands.
+- Do not assume Excel, LibreOffice, Python libraries, Node packages, or Google APIs are available.
+- Do not install dependencies unless the user explicitly asks and approves.
+
+## Final response guidance
+
+When delivering spreadsheet content inline, make it easy to copy into a sheet. When files are created or edited, include:
+
+- The file path.
+- The sheet/table structure.
+- Key formulas or assumptions.
+- Validation performed or remaining risks.

package/server/skills.mjs

@@ -25,6 +25,10 @@ function normalizeString(value) {
   return typeof value === 'string' && value.trim() ? value.trim() : undefined
 }
 
+function normalizeSkillName(value) {
+  return normalizeString(value)?.toLowerCase()
+}
+
 function normalizeStringArray(value) {
   if (!Array.isArray(value)) return []
   const result = []
@@ -39,7 +43,16 @@ function normalizeStringArray(value) {
 }
 
 export function normalizeSkillNames(value) {
-  return normalizeStringArray(value).filter(isValidSkillName)
+  if (!Array.isArray(value)) return []
+  const result = []
+  const seen = new Set()
+  for (const item of value) {
+    const name = normalizeSkillName(item)
+    if (!name || !isValidSkillName(name) || seen.has(name)) continue
+    seen.add(name)
+    result.push(name)
+  }
+  return result
 }
 
 function isPathInside(root, target) {
@@ -234,10 +247,10 @@ function skillFromStandardMarkdown(rootDir, source, text) {
   if (!parsed?.body) return null
 
   const rawManifest = parseSimpleYamlMap(parsed.frontmatter)
-  const name = normalizeString(rawManifest.name)
+  const name = normalizeSkillName(rawManifest.name)
   const description = normalizeString(rawManifest.description)
   if (!name || !isValidSkillName(name)) return null
-  if (name !== path.basename(rootDir)) return null
+  if (name !== normalizeSkillName(path.basename(rootDir))) return null
   if (!description || description.length > 1024) return null
 
   const metadata = normalizeMetadata(rawManifest.metadata)
@@ -267,7 +280,7 @@ async function loadLegacySkillDirectory(rootDir, source) {
   if (!rawManifest || typeof rawManifest !== 'object') return null
   if (rawManifest.enabled === false) return null
 
-  const name = normalizeString(rawManifest.name) || path.basename(rootDir)
+  const name = normalizeSkillName(rawManifest.name) || normalizeSkillName(path.basename(rootDir))
   if (!isValidSkillName(name)) return null
 
   const entry = normalizeString(rawManifest.entry) || defaultEntry
@@ -469,13 +482,15 @@ export async function listSkillSummaries() {
 }
 
 export async function findGlobalSkill(name) {
+  const skillName = normalizeSkillName(name)
   const skills = await loadGlobalSkills()
-  return skills.find((skill) => skill.name === name) || null
+  return skills.find((skill) => skill.name === skillName) || null
 }
 
 export async function findProjectSkill(name, workspaceRoot) {
+  const skillName = normalizeSkillName(name)
   const skills = await loadProjectSkills(workspaceRoot)
-  return skills.find((skill) => skill.name === name) || null
+  return skills.find((skill) => skill.name === skillName) || null
 }
 
 export async function findSkill(name) {

package/server/tools/index.mjs

@@ -10,6 +10,7 @@ import {
   loadSelectedGlobalSkills,
   loadSelectedProjectSkills,
   mergeSkills,
+  normalizeSkillNames,
   readSkillResource,
 } from '../skills.mjs'
 import { getToolWorkspaceRoot } from '../utils/workspace.mjs'
@@ -571,7 +572,7 @@ function activeSkillsForContext(context) {
 }
 
 function activeSkillByName(context, name) {
-  const skillName = String(name || '')
+  const skillName = normalizeSkillNames([name])[0]
   return activeSkillsForContext(context).find((skill) => skill.name === skillName)
 }