@aryacode/dsh-engineering 0.4.22 → 0.6.0

package/CHANGELOG.md

@@ -1,5 +1,103 @@
 # Changelog
 
+## [0.6.0] - 2026-10-27
+
+### Added
+
+- **FactoryTalk Optix NetLogic reference.** The bundle vendors a pinned snapshot of Rockwell's
+  [NetLogic cheat sheet](https://github.com/FactoryTalk-Optix/NetLogic_CheatSheet) (51 pages, commit
+  `5e652541`) and exposes it as one tool instead of dumping C# into the prompt:
+  `optix_netlogic` with no arguments returns the contents, a `query` ranks pages by title, topic and
+  symbol before body text, and a `slug` returns that page verbatim. Every page result carries its
+  `risk` (`normal`, `advanced` or `unsupported`, with the repository's own Advanced section marked
+  unsupported), the upstream commit, a source URL and the upstream disclaimer that the snippets may use
+  non-public APIs and must not be used in production as-is.
+- **A FactoryTalk Optix skill.** The row previously had tools and no guidance; `factorytalk-optix` now
+  names the surface, the NetLogic reader and its risk rule, and the report-before-claiming boundary.
+- The `optix` row gains a `netlogic` switch (on by default). Off removes one tool and the guidance
+  without touching the prebuilt engine.
+- `scripts/sync-netlogic.mjs` is the only writer of the snapshot: it pins a commit, copies
+  `pages/*.md`, and generates `index.json` plus `SOURCE.json` (commit, page count, risk counts).
+  `pnpm netlogic:sync` refreshes it; `test/netlogic.spec.mjs` fails when the index and the pages drift.
+
+## [0.5.0] - 2026-10-27
+
+Commissioned against the Laminator fixtures: a real Studio 5000 ACD exported to a 5.4 MB L5X, a
+real FactoryTalk View ME export, and two MotionWorks projects.
+
+### Fixed — FactoryTalk View could not read a real Studio export
+
+`ftview_open` reported **0 displays and 0 tags** for a genuine ME export. Five independent causes:
+
+- **The tag CSV delimiter was hard-coded to `;`** while Studio exports are comma-separated and use a
+  leading `;` only as a comment prefix, so every data row parsed as one cell. The separator is now
+  detected per line with quote awareness.
+- **The header was assumed to follow the `###002` line.** A real export writes the header first, then
+  `###002`, then `;Folders Section` / `;Tag Section` comments, and the parser took the first comment
+  for the header. It now finds a `Tag Name` header on either side of the version line.
+- **Short rows were rejected.** Studio omits trailing empty columns, so 29-column headers come with
+  23-column rows. Only over-long rows are errors now.
+- **`<gfx>` displays were refused.** ME display XML is `<gfx>` (Gfx-ME17.xsd), not `<display>`.
+- **UTF-16 files were decoded as UTF-8**, producing a root element of `\0g\0f\0x`; BOM sniffing now
+  selects UTF-16LE.
+
+Also normalised the host contract: `scanApplication` accepts the `FsTarget` that `fs.resolve`
+returns instead of calling string methods on it.
+
+Verified on `Laminator/HMI/Main`: **18 displays, 361 tags, 0 parse errors** (was 0 and 0). Reading a
+`<gfx>` export is supported; the staged edit tools were built for the `<display>` shape and are not
+verified to rewrite `<gfx>` objects.
+
+### Fixed — Studio 5000 checks drowned real findings in false positives
+
+`run_checks` on the Laminator export produced 562 findings, 315 of them "unresolved reference", on a
+project that compiles. Every one was an alias to module I/O (`In_FF_ModulePresent` →
+`FanFold_StraightCut_AENTR:2:I.7`): aliases carry no `DataType`, their targets are not declared
+`<Tag>`s, and the fast resolver stopped instead of falling back to the canonical one.
+
+- `resolveSymbol` now falls back to `resolveTag`, and an empty type with a trailing numeric part
+  resolves as a bit.
+- A member of a module-defined or opaque type (`Drive_MCFG.CommFLT`, `Axis_MAH.PC`) and `S:`/`R:`
+  system flags are reported as coverage limits, not unresolved references.
+
+**562 → 264 findings; unresolved 315 → 16**, with the unused-tag and multiple-write findings intact.
+
+### Fixed — Studio 5000 edits
+
+- **`add_tag` required a value for a scalar tag.** Studio defaults scalars to zero, so declaring a
+  plain REAL was impossible without inventing a value; `value` is now optional and still validated
+  when supplied.
+- **Live reads refused aliases.** `read_live_tags` and `poll_live_tags` ran the write rule, so
+  ordinary module-point BOOLs were rejected as "not a writable declared scalar". Reads and polls now
+  take an alias's type from the canonical resolver; writes still refuse an alias and name the tag to
+  write instead.
+- **`export_acd_to_l5x` failed when Studio had the ACD open.** The SDK opens the file itself, so the
+  normal case — the project the user is working on — failed with "being used by another process". The
+  export now copies the ACD and opens the copy, with the source hash re-checked before and after.
+
+Verified end to end on the Laminator export: `draft_tag` (REAL, no value) plus a rung replacement
+validated and applied to a 5,391,840-byte L5X.
+
+### Fixed — MotionWorks IEC
+
+- **`mw_get kind="block"` crashed with `KeyError: 'native_libraries'`.** The block interface is decoded
+  against the installed libraries as well as the saved project; the read service now supplies the live
+  inventory and falls back to the project's own registry when no IDE answers.
+- **`mw_edit_variables` refused with "Live declarations disagree with saved baseline"** and nothing
+  else. The refusal now names the missing, unexpected and changed declarations with their live and
+  saved values.
+
+Verified on both Laminator projects through the shipped engine: attach into
+`backups/motionworks/<project>/<stamp>/`, then `workflow_check`, `pous`, `read_text`, `validate`,
+`tasks`, `types`, `globals` and `library` all succeed (8 POUs, 5 tasks, ~170 globals, 297 types,
+23 blocks each).
+
+### Documentation
+
+- `docs/optix-netlogic.md`: how the FactoryTalk Optix NetLogic cheat sheet should be carried — a
+  pinned snapshot, a generated index, and one `optix_netlogic` retrieval tool, with the risk flags and
+  disclaimers the repository itself requires.
+
 ## [0.4.22] - 2026-10-27
 
 ### Fixed

package/README.md

@@ -4,7 +4,7 @@
 MotionWorks IEC and FactoryTalk Optix sit behind a single install, one row per surface. A row is the
 switch: disable it and that plugin unloads, so its tool schemas leave the request.
 
-`@aryacode/dsh-engineering` · 84 tools · 5 rows · 4 products · MIT
+`@aryacode/dsh-engineering` · 85 tools · 5 rows · 4 products · MIT
 
 ---
 
@@ -39,7 +39,7 @@ bundle.
 | **Studio 5000 L5X** | 40 | all on | Node for offline reads; Studio 5000 + Logix Designer SDK for SDK work; a controller for live |
 | **FactoryTalk View ME** | 14 | all on | Node for an export folder; FactoryTalk View Studio ME for discovery and the handoff |
 | **MotionWorks IEC** | 24 | all on | MotionWorks IEC 3 Pro installed; verified against 1.19 |
-| **FactoryTalk Optix** | 9 (3 need preview) | all on | Optix Studio installed; Tesseract for OCR |
+| **FactoryTalk Optix** | 10 (3 need preview) | all on | Optix Studio installed; Tesseract for OCR |
 
 Tool groups, by product:
 
@@ -47,7 +47,7 @@ Tool groups, by product:
   Designer SDK, 7 supervised controller, 7 live EtherNet/IP (2 reads, 2 identity/discovery, 1 batch read, 2 approval-gated writes)
 - **FactoryTalk View** — 6 read, 5 staged edits, 3 Studio handoff
 - **MotionWorks IEC** — 7 session, 7 saved-project reads, 3 verify, 7 direct edits
-- **Optix** — 6 tools, unlinked to a skill until one exists
+- **Optix** — 10 tools, linked to the `factorytalk-optix` guidance; `optix_netlogic` reads the pinned NetLogic cheat-sheet snapshot, and the 9 bridge tools need Studio
 
 ## Switches
 
@@ -62,7 +62,7 @@ down:
 | `ftview` | FactoryTalk View ME read, guidance and edits; optionally the Studio handoff | `edit` (on), `studio` (on) |
 | `mwi` | MotionWorks IEC session, reads, verify, guidance and the edits | `workspace` (empty), `edit` (on) |
 | `viewers` | ladder + ME display previews | — |
-| `optix` | FactoryTalk Optix Studio authoring | `bridgeUrls` (8768), `bridgeToken` (empty), `allowWrites` (on), `allowDestructive` (on), `allowInvoke` (on), `preview` (on), `allowPreviewInput` (on), `cdpUrl`, `previewUrl`, `ocrExecutable` (empty), `ocrLanguage` (`eng`), `ocrTessdataDirectory` (empty) |
+| `optix` | FactoryTalk Optix Studio authoring and the NetLogic reader | `bridgeUrls` (8768), `bridgeToken` (empty), `allowWrites` (on), `allowDestructive` (on), `allowInvoke` (on), `preview` (on), `allowPreviewInput` (on), `cdpUrl`, `previewUrl`, `ocrExecutable` (empty), `ocrLanguage` (`eng`), `ocrTessdataDirectory` (empty), `netlogic` (on) |
 
 **Why the row list collapsed.** The Plugins page lists a bundle's rows flat and skips `cordis:group`
 containers (`plugin-manager/src/index.ts`: *"Group rows are containers … their children carry the
@@ -91,7 +91,9 @@ Where each surface actually stands, based on what has been exercised.
   downloads accepting only a verification id produced by a passed build
 - Live EtherNet/IP: polling, one-request batch reads of up to 64 tags, controller identity and clock, LAN discovery, and a bounded trend
 - Supervised live writes: `write_live_tag` and `write_controller_clock` need the row's `allowWrites` switch **and** per-call human approval, and report the value the controller reads back
-- FactoryTalk View ME: export reads, structural checks, comparison, staged display and HMI tag edits
+- FactoryTalk View ME: export reads, structural checks, comparison, staged display and HMI tag edits;
+  reads verified on a real Studio export (`<gfx>` displays and the comma-delimited tag CSV —
+  18 displays, 361 tags, no parse errors)
 - MotionWorks IEC: verified live against 3 Pro 1.19 — direct open with a verified pre-edit backup
   under `.motionworks/attached`, attach-without-reload to a project the IDE already has open, start,
   worksheet, close, saved-project reads, compile diagnostics, code replacement, declaration add, POU
@@ -110,7 +112,7 @@ the lean profile is the one that turns them off.
 - FBD/SFC semantics are not exercised
 - FactoryTalk View Studio work is a **manual handoff**: the bundle plans the import and preflights
   changed files, it does not import
-- Optix has no skill yet; its tools follow the `optix` row config and the Studio bridge
+- Optix is linked to a skill now: the `factorytalk-optix` guidance plus the NetLogic reader; the bridge tools still follow the `optix` row config and the Studio bridge
 
 **Not in this repository** — CAD mounts as an independent plugin.
 
@@ -118,6 +120,7 @@ the lean profile is the one that turns them off.
 
 | Version | What landed |
 |---|---|
+| **0.5.0** | Commissioned on real files: FactoryTalk View reads genuine Studio exports (`<gfx>` displays, comma CSVs, UTF-16) instead of reporting zero; L5X alias resolution cuts 315 false "unresolved reference" findings to 16 on a 1279-tag project; `add_tag` no longer demands a value; a locked ACD exports from a copy |
 | **0.4.17** | MotionWorks edits the real project again: the pre-edit backup is written where the bridge and engine verify it, an already-open project is attached instead of reloaded, and `mw_get` returns the body hash the edit takes |
 | **0.4.15** | Every tool surface ships on by default; the row Configure pages switch them off |
 | **0.4.4** | One row per product: five rows, optional surfaces moved into each row's config |
@@ -172,7 +175,7 @@ even when the implementation behind a row is replaced.
 | `ftview` | `dsh-engineering-ftview` | `edit` on · `studio` on |
 | `mwi` | `dsh-engineering-mwi` | `edit` on |
 | `viewers` | `@aryacode/dsh-engineering` (its own client face: ladder + ME display previews) | — |
-| `optix` | `@aryacode/dsh-engineering/optix` (wraps the prebuilt `@aryacode/dsh-tool-ft-optix`) | `allowWrites` on · `allowDestructive` on · `allowInvoke` on · `preview` on |
+| `optix` | `@aryacode/dsh-engineering/optix` (wraps the prebuilt `@aryacode/dsh-tool-ft-optix`) | `allowWrites` on · `allowDestructive` on · `allowInvoke` on · `preview` on · `netlogic` on |
 
 ## Safety
 

package/cordis.patch.yml

@@ -9,7 +9,7 @@
 #   l5x    → editing, studio, live, allowWrites   (all on by default)
 #   ftview → edit, studio                          (both on by default)
 #   mwi    → edit                                  (on by default)
-#   optix  → authoring, destructive, invoke, preview (all on by default)
+#   optix  → authoring, destructive, invoke, preview, netlogic (all on by default)
 #
 # Every surface ships on so a fresh install can use the full plugin. Turn one
 # off from the row's Configure page when the profile is read-only or the request

package/entries/netlogic.js

@@ -0,0 +1,266 @@
+/**
+ * FactoryTalk Optix NetLogic retrieval.
+ *
+ * The vendor repository (FactoryTalk-Optix/NetLogic_CheatSheet) is vendored as a
+ * pinned snapshot by `scripts/sync-netlogic.mjs`. This entry exposes it as one tool
+ * so a model can pull the page it needs instead of every page at once: the table of
+ * contents costs a few hundred tokens, a page a few thousand.
+ *
+ * Every answer carries the page's risk flag and the upstream disclaimer. The
+ * repository itself says the snippets may use non-public APIs, may break a project
+ * and are not for production as-is; this module must not outrank that.
+ *
+ * @module dsh-engineering/entries/netlogic
+ */
+
+import { existsSync, readFileSync } from 'node:fs'
+import { dirname, join } from 'node:path'
+import { fileURLToPath } from 'node:url'
+
+/** Plugin name in a composition. */
+export const name = 'engineering-optix-netlogic'
+
+/** The tool needs the registry; the skill is registered when the host offers it. */
+export const inject = ['tools']
+
+/** Registration metadata, kept here so a test can assert it without the file. */
+export const SKILL = {
+  name: 'factorytalk-optix',
+  description: 'FactoryTalk Optix Studio engineering: inspect and author the loaded project through the Studio bridge, validate expressions, take preview screenshots, read NetLogic C# snippets from the pinned cheat-sheet snapshot, and start from a template.',
+  whenToUse: 'The user asks about FactoryTalk Optix — project structure, nodes, variables, aliases, expressions, NetLogic C#, alarms, trends, recipes, OPC UA, or the emulator preview. Route FactoryTalk View ME elsewhere.',
+}
+
+/** The upstream statement every page result repeats. */
+export const DISCLAIMER = 'Reference only. The upstream repository states these snippets may use non-public APIs, may break a project, and must not be used in production as-is; adapt and test them in your own project.'
+
+const HERE = dirname(fileURLToPath(import.meta.url))
+const DATA_DIRS = [
+  join(HERE, '..', 'vendor', 'optix', 'netlogic'),
+  join(HERE, '..', 'packages', 'optix', 'netlogic'),
+]
+const SKILL_PATHS = [
+  join(HERE, '..', 'vendor', 'optix', 'SKILL.md'),
+  join(HERE, '..', 'packages', 'optix', 'SKILL.md'),
+]
+
+const RISKS = new Set(['normal', 'advanced', 'unsupported'])
+
+/** The first existing path from a candidate list. */
+function first(candidates) {
+  for (const candidate of candidates) if (existsSync(candidate)) return candidate
+  return undefined
+}
+
+/** The vendored snapshot directory, or undefined when it was not shipped. */
+export function dataDirectory() {
+  return first(DATA_DIRS)
+}
+
+/** Cached index and source pin; the snapshot is immutable for a process. */
+let cachedSnapshot
+function snapshot() {
+  if (cachedSnapshot === undefined) cachedSnapshot = loadSnapshot()
+  return cachedSnapshot
+}
+
+/** Cached page text, keyed by slug. */
+const pageCache = new Map()
+
+/** Read the generated index and source pin. */
+export function loadSnapshot() {
+  const directory = dataDirectory()
+  if (directory === undefined) throw new Error('The NetLogic snapshot is missing; run `pnpm netlogic:sync` before packing.')
+  const index = JSON.parse(readFileSync(join(directory, 'index.json'), 'utf8'))
+  const source = JSON.parse(readFileSync(join(directory, 'SOURCE.json'), 'utf8'))
+  if (!Array.isArray(index) || index.length === 0) throw new Error('The NetLogic index is empty; run `pnpm netlogic:sync`.')
+  for (const row of index) {
+    if (typeof row?.slug !== 'string' || !/^[a-z0-9][a-z0-9-]*$/u.test(row.slug)) throw new Error(`The NetLogic index has an invalid slug: ${String(row?.slug)}`)
+    if (!RISKS.has(row.risk)) throw new Error(`The NetLogic index has an invalid risk for ${row.slug}: ${String(row.risk)}`)
+  }
+  return { directory, index, source }
+}
+
+/** Read one page's markdown, refusing anything outside the snapshot. */
+export function pageOf(slug) {
+  if (typeof slug !== 'string' || !/^[a-z0-9][a-z0-9-]*$/u.test(slug)) {
+    throw Object.assign(new Error(`"${String(slug)}" is not a page slug.`), { code: 'INVALID_ARGUMENT' })
+  }
+  const cached = pageCache.get(slug)
+  if (cached !== undefined) return cached
+  const { directory } = snapshot()
+  const path = join(directory, 'pages', `${slug}.md`)
+  if (!existsSync(path)) throw Object.assign(new Error(`No page named "${slug}".`), { code: 'COMPONENT_NOT_FOUND' })
+  const markdown = readFileSync(path, 'utf8')
+  pageCache.set(slug, markdown)
+  return markdown
+}
+
+/** Score one page against a query; higher is better, zero means no hit. */
+function scoreOf(row, query) {
+  const needle = query.toLowerCase()
+  if (row.slug === needle) return 100
+  if (row.title.toLowerCase() === needle) return 90
+  if (row.symbols.some(symbol => symbol.toLowerCase() === needle)) return 80
+  if (row.slug.includes(needle)) return 60
+  if (row.title.toLowerCase().includes(needle)) return 55
+  if (row.topics.some(topic => topic === needle)) return 45
+  if (row.symbols.some(symbol => symbol.toLowerCase().startsWith(needle))) return 35
+  return 0
+}
+
+/** Rank pages by metadata, then by body text for the remainder. */
+export function search(index, query, limit = 8) {
+  const scored = index.map(row => ({ row, score: scoreOf(row, query) })).filter(hit => hit.score > 0)
+  if (scored.length < limit) {
+    const seen = new Set(scored.map(hit => hit.row.slug))
+    for (const row of index) {
+      if (seen.has(row.slug)) continue
+      const body = pageOf(row.slug)
+      if (body.toLowerCase().includes(query.toLowerCase())) scored.push({ row, score: 20 })
+    }
+  }
+  return scored
+    .sort((left, right) => right.score - left.score || left.row.title.localeCompare(right.row.title))
+    .slice(0, limit)
+    .map(hit => ({ slug: hit.row.slug, title: hit.row.title, section: hit.row.section, risk: hit.row.risk, score: hit.score }))
+}
+
+/** The coverage every answer carries. */
+const COVERAGE = {
+  complete: false,
+  unverified: [
+    'C# compiles against the installed Optix version',
+    'the snippet is safe for this project',
+    'runtime behaviour in the emulator or on the panel',
+  ],
+}
+
+/**
+ * Answer one call.
+ *
+ * @param args - `slug` for one page, `query` to search, or neither for the contents.
+ * @returns The tool envelope.
+ */
+export function answer(args = {}) {
+  const { index, source } = snapshot()
+  const slug = typeof args.slug === 'string' && args.slug.trim() !== '' ? args.slug.trim() : undefined
+  if (slug !== undefined) {
+    const markdown = pageOf(slug)
+    const row = index.find(entry => entry.slug === slug)
+    return {
+      summary: `Read ${row.title} (${row.risk}) from ${source.repository}@${source.commit.slice(0, 8)}.`,
+      data: {
+        slug,
+        title: row.title,
+        section: row.section,
+        risk: row.risk,
+        commit: source.commit,
+        source_url: `https://github.com/${source.repository}/blob/${source.commit}/pages/${slug}.md`,
+        code_blocks: Math.floor((markdown.match(/^```/gmu) ?? []).length / 2),
+        disclaimer: DISCLAIMER,
+        markdown,
+      },
+      next: [{ tool: 'optix_edit', why: 'Apply the adapted snippet through the Studio bridge, then validate.' }],
+      coverage: COVERAGE,
+    }
+  }
+  const query = typeof args.query === 'string' && args.query.trim() !== '' ? args.query.trim() : undefined
+  if (query === undefined) {
+    return {
+      summary: `${index.length} NetLogic pages from ${source.repository}@${source.commit.slice(0, 8)}.`,
+      data: {
+        commit: source.commit,
+        disclaimer: DISCLAIMER,
+        pages: index.map(row => ({ slug: row.slug, title: row.title, section: row.section, risk: row.risk })),
+      },
+      next: [{ tool: 'optix_netlogic', args: { query: 'sessions' }, why: 'Read the page for a topic before writing NetLogic.' }],
+      coverage: COVERAGE,
+    }
+  }
+  const requested = typeof args.limit === 'number' && Number.isInteger(args.limit) ? args.limit : 8
+  const limit = Math.min(Math.max(requested, 1), 20)
+  const hits = search(index, query, limit)
+  return {
+    summary: hits.length === 0
+      ? `No NetLogic page matches "${query}".`
+      : `${hits.length} NetLogic page(s) match "${query}".`,
+    data: { commit: source.commit, query, hits, ...(hits.length === 0 ? { suggestion: 'Call optix_netlogic with no arguments for the contents, then pass one slug.' } : {}) },
+    next: hits.length === 0
+      ? [{ tool: 'optix_netlogic', why: 'List the contents and pick a slug.' }]
+      : [{ tool: 'optix_netlogic', args: { slug: hits[0].slug }, why: 'Read the best-matching page.' }],
+    coverage: COVERAGE,
+  }
+}
+
+/** The failure envelope every engineering tool returns. */
+function failure(error) {
+  const code = typeof error?.code === 'string' ? error.code : 'INTERNAL'
+  return {
+    error: {
+      code,
+      message: error instanceof Error ? error.message : String(error),
+      fix: code === 'COMPONENT_NOT_FOUND'
+        ? 'Call optix_netlogic with no arguments for the available slugs.'
+        : 'Correct the argument and retry; the snapshot is local and needs no network.',
+      candidates: [],
+    },
+  }
+}
+
+/** The tool definition this entry registers. */
+export const definition = {
+  name: 'optix_netlogic',
+  description: 'Read the pinned FactoryTalk Optix NetLogic cheat-sheet snapshot: return the contents with no arguments, ranked pages for a query, or one page in full for a slug. Every page result carries its risk flag, the upstream commit and the upstream disclaimer.',
+  parameters: {
+    type: 'object',
+    properties: {
+      query: { type: 'string', description: 'Topic, method or symbol to search for, for example "sessions", "PeriodicTask" or "alarm".' },
+      slug: { type: 'string', description: 'Exact page slug from the contents, for example "netlogic-lifecycle".' },
+      limit: { type: 'integer', description: 'Most search hits to return; defaults to 8, maximum 20.', minimum: 1, maximum: 20 },
+    },
+    additionalProperties: false,
+    required: [],
+  },
+  output: {
+    schema: { type: 'object', additionalProperties: true },
+    render(_args, value) {
+      return [{ type: 'text', text: JSON.stringify(value, undefined, 2) }]
+    },
+  },
+  async execute(args) {
+    try {
+      return answer(args ?? {})
+    } catch (error) {
+      return failure(error)
+    }
+  },
+}
+
+/**
+ * Register the retrieval tool and, when the host offers it, the Optix skill.
+ *
+ * @param ctx - Plugin context with the tool registry and an optional skills service.
+ */
+export function apply(ctx) {
+  ctx.tools.register(definition)
+  const skillPath = first(SKILL_PATHS)
+  const skills = typeof ctx.get === 'function' ? ctx.get('skills') : undefined
+  if (skillPath === undefined || skills === undefined || typeof skills.register !== 'function') {
+    try {
+      ctx.logger?.warn?.('optix-netlogic: the skill was not registered; the tool is available.')
+    } catch {
+      // A logger that refuses must not turn a missing skill into a crash.
+    }
+    return
+  }
+  const raw = readFileSync(skillPath, 'utf8')
+  const content = raw.replace(/^---\r?\n[\s\S]*?\r?\n---\r?\n/u, '')
+  skills.register({
+    ...SKILL,
+    content,
+    path: skillPath,
+    source: 'custom',
+    provider: '@aryacode/dsh-engineering',
+    invocation: { modelInvocable: true, userInvocable: true },
+  })
+}

package/entries/optix.js

@@ -11,6 +11,7 @@
 import z from '@deepseek-ai/schemastery'
 import { dirname, join } from 'node:path'
 import * as optix from '../vendor/optix/lib/index.js'
+import * as netlogic from './netlogic.js'
 import { flag, list, text } from './config.js'
 
 export const name = 'engineering-optix'
@@ -53,6 +54,9 @@ export const Config = z.object({
   ocrTessdataDirectory: z.string().default('').volatile().description(
     'Folder holding the .traineddata files. Empty derives tessdata beside tesseract.exe.',
   ),
+  netlogic: z.boolean().default(true).volatile().description(
+    'Register the pinned NetLogic cheat-sheet reader (optix_netlogic) and the FactoryTalk Optix skill. Off removes one tool and its guidance.',
+  ),
   requestTimeoutMs: z.natural().min(1).max(2147483647).default(20000),
   maxResponseBytes: z.natural().min(1).default(2000000),
   maxBatchOps: z.natural().min(1).default(40),
@@ -93,12 +97,21 @@ export function apply(ctx, config = {}) {
     }
   }
   let mounted
+  let netlogicMounted
   // Volatile-only config changes are committed into the live references and
   // emit `loader/volatile-update` instead of re-running apply(). The prebuilt
   // plugin reads its options once, so remount it to pick up a switch change.
   const sync = () => {
     if (mounted !== undefined) void mounted.dispose()
     mounted = ctx.plugin(optix, options())
+    // The NetLogic reader is this repository's own entry: it is a separate plugin
+    // so the row's `netlogic` switch can drop one tool and its guidance without
+    // touching the prebuilt engine.
+    if (netlogicMounted !== undefined) {
+      void netlogicMounted.dispose()
+      netlogicMounted = undefined
+    }
+    if (flag(config.netlogic)) netlogicMounted = ctx.plugin(netlogic)
   }
   sync()
   ctx.on('loader/volatile-update', sync)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aryacode/dsh-engineering",
-  "version": "0.4.22",
+  "version": "0.6.0",
   "description": "AryaCode engineering bundle: one install surface for the Studio 5000 L5X, FactoryTalk View ME, MotionWorks IEC and FactoryTalk Optix surfaces, one row per product, with the ladder and display previews.",
   "keywords": [
     "dsh",
@@ -56,9 +56,10 @@
     }
   },
   "scripts": {
-    "test": "node --import tsx --test test/patch.spec.mjs test/docs.spec.mjs test/tool-reference.spec.mjs test/manifests.spec.mjs test/client-face.spec.mjs test/config.spec.mjs",
+    "test": "node --import tsx --test test/patch.spec.mjs test/docs.spec.mjs test/tool-reference.spec.mjs test/manifests.spec.mjs test/client-face.spec.mjs test/config.spec.mjs test/netlogic.spec.mjs",
     "docs:tools": "tsx scripts/gen-tool-reference.ts",
     "docs:check": "tsx scripts/gen-tool-reference.ts --check",
+    "netlogic:sync": "node scripts/sync-netlogic.mjs",
     "build:client": "node scripts/build-client.mjs",
     "assemble": "node scripts/assemble.mjs",
     "prepack": "node scripts/build-client.mjs && node scripts/assemble.mjs"

package/vendor/ftview/SKILL.md

@@ -13,6 +13,12 @@ A staged edit is written to a NEW export folder; the source is never modified. T
 a manual import for a human; it does not drive Studio, contact a terminal, or decode MER or MED
 databases.
 
+Both export shapes are read: the Studio ME export (`<gfx>` displays, `Gfx-ME17.xsd`, a comma-delimited
+`Tags/*.csv` with its header before the `###002` line and `;` comment rows) and the older
+`<display>`-root files. Staged attribute edits were built for the `<display>` shape; on a `<gfx>`
+export, read and verify with this surface and make the change in Studio, or confirm the edit applied
+before relying on it.
+
 ## Backups
 
 Every engineering product backs up into the same tree, so a backup means one place whichever surface

package/vendor/ftview/lib/model.d.ts

@@ -8,7 +8,7 @@
  *
  * @module dsh-engineering-ftview/model
  */
-import { type FsService } from './core.ts';
+import { type FsService, type FsTarget } from './core.ts';
 /** One row of the HMI tag CSV, keyed by its canonical columns. */
 export interface TagRow {
     readonly name: string;
@@ -51,11 +51,16 @@ export interface TagCsv {
     readonly rows: readonly TagRow[];
     readonly errors: readonly string[];
 }
-/** Split one semicolon-separated CSV line, honouring double quotes. */
+/** Split one CSV line, honouring double quotes and the line's own separator. */
 export declare function splitCsv(line: string): string[];
 /**
  * Parse the HMI tag CSV.
  *
+ * The ME export writes an optional header first, then a `###002` version line, then
+ * `;` comment/section lines and the data rows. Earlier builds assumed the header
+ * followed the version line and treated `;Folders Section` as the header, so a real
+ * export parsed as zero tags.
+ *
  * @param text - Whole CSV file.
  * @returns Rows plus structural errors; a file with no `###002` marker yields no rows.
  */
@@ -77,4 +82,4 @@ export declare function parseDisplay(file: string, text: string): DisplayFile;
  * @param maxFiles - Largest number of display/tag files accepted.
  * @returns The scanned application.
  */
-export declare function scanApplication(fs: FsService, root: string, signal: AbortSignal | undefined, maxFiles: number): Promise<Application>;
+export declare function scanApplication(fs: FsService, root: string | FsTarget, signal: AbortSignal | undefined, maxFiles: number): Promise<Application>;