@platformos/platformos-check-common 0.0.7 → 0.0.9

package/src/context-utils.ts

@@ -1,15 +1,48 @@
-import { URI, Utils } from 'vscode-uri';
-import { AbstractFileSystem, FileTuple, FileType, UriString } from '@platformos/platformos-common';
-import { parseJSON } from './json';
-import { join } from './path';
+import { load } from 'js-yaml';
 import {
-  MetafieldCategory,
-  MetafieldDefinitionMap,
-  SourceCodeType,
-  Theme,
-  Translations,
-} from './types';
-import { isError } from './utils';
+  AbstractFileSystem,
+  FileTuple,
+  FileType,
+  TranslationProvider,
+  UriString,
+} from '@platformos/platformos-common';
+import { URI } from 'vscode-uri';
+import { join } from './path';
+import { SourceCodeType, App, Translations } from './types';
+
+/**
+ * Returns a function that loads and merges ALL translation files for a given
+ * locale within a specific translations base directory URI
+ * (e.g. `file:///app/translations` or
+ * `file:///modules/common-styling/public/translations`).
+ *
+ * Covers both the single-file layout (`{base}/{locale}.yml`) and the split-file
+ * layout (`{base}/{locale}/*.yml`).
+ *
+ * Only files whose first YAML key matches `locale` are included — this mirrors
+ * how platformOS determines a file's locale from its content, not its path.
+ *
+ * In-memory editor buffers take precedence over the filesystem so that unsaved
+ * changes are reflected immediately.
+ */
+export const makeGetTranslationsForBase = (fs: AbstractFileSystem, app: App) => {
+  const provider = new TranslationProvider(fs);
+  const cache = new Map<string, Promise<Translations>>();
+
+  return (translationBaseUri: string, locale: string): Promise<Translations> => {
+    const key = `${translationBaseUri}::${locale}`;
+    if (!cache.has(key)) {
+      const contentOverride = (uri: string): string | undefined =>
+        app.find((sc) => sc.type === SourceCodeType.YAML && sc.uri === uri)?.source;
+
+      cache.set(
+        key,
+        provider.loadAllTranslationsForBase(URI.parse(translationBaseUri), locale, contentOverride),
+      );
+    }
+    return cache.get(key)!;
+  };
+};
 
 export type FileExists = (uri: string) => Promise<boolean>;
 
@@ -33,71 +66,49 @@ export const makeFileSize = (fs: AbstractFileSystem) =>
     }
   };
 
-export const makeGetDefaultLocaleFileUri = getDefaultLocaleFileUriFactoryFactory('default.json');
-export const makeGetDefaultSchemaLocaleFileUri =
-  getDefaultLocaleFileUriFactoryFactory('.default.schema.json');
-function getDefaultLocaleFileUriFactoryFactory(postfix = '.default.json') {
-  return function getDefaultLocaleFileUriFactory(fs: AbstractFileSystem) {
-    return (rootUri: string) => getDefaultLocaleFile(fs, rootUri, postfix);
-  };
-}
+export const makeGetDefaultLocaleFileUri = (fs: AbstractFileSystem) => (rootUri: string) =>
+  getDefaultLocaleFile(fs, rootUri);
 
-export const makeGetDefaultLocale = getDefaultLocaleFactoryFactory('.default.json');
-export const makeGetDefaultSchemaLocale = getDefaultLocaleFactoryFactory('.default.schema.json');
-function getDefaultLocaleFactoryFactory(postfix = '.default.json') {
-  return function getDefaultLocaleFactory(fs: AbstractFileSystem, rootUri: string) {
-    return cached(() => getDefaultLocale(fs, rootUri, postfix));
-  };
-}
+export const makeGetDefaultLocale = (fs: AbstractFileSystem, rootUri: string) =>
+  cached(() => getDefaultLocale(fs, rootUri));
 
-export const makeGetDefaultTranslations = getDefaultTranslationsFactoryFactory('.default.json');
-export const makeGetDefaultSchemaTranslations =
-  getDefaultTranslationsFactoryFactory('.default.schema.json');
-// prettier-ignore
-function getDefaultTranslationsFactoryFactory(postfix = '.default.json') {
-  return function getDefaultTranslationsFactory(fs: AbstractFileSystem, theme: Theme, rootUri: string) {
-    return cached(() => getDefaultTranslations(fs, theme, rootUri, postfix));
-  };
-}
+export const makeGetDefaultTranslations = (fs: AbstractFileSystem, app: App, rootUri: string) =>
+  cached(() => getDefaultTranslations(fs, app, rootUri));
 
 async function getDefaultLocaleFile(
   fs: AbstractFileSystem,
   rootUri: string,
-  postfix = '.default.json',
-) {
-  const files = await fs.readDirectory(join(rootUri, 'locales'));
-  return files.find(([uri]) => uri.endsWith(postfix))?.[0];
-}
-
-async function getDefaultLocale(
-  fs: AbstractFileSystem,
-  rootUri: string,
-  postfix: string,
-): Promise<string> {
+): Promise<string | undefined> {
+  const enYmlUri = join(rootUri, 'app/translations/en.yml');
   try {
-    const defaultLocaleFile = await getDefaultLocaleFile(fs, rootUri, postfix);
-    if (!defaultLocaleFile) return 'en';
-    const defaultLocaleFileName = Utils.basename(URI.parse(defaultLocaleFile));
-    return defaultLocaleFileName.split('.')[0];
-  } catch (error) {
-    console.error(error);
-    return 'en';
+    await fs.stat(enYmlUri);
+    return enYmlUri;
+  } catch {
+    return undefined;
   }
 }
 
+async function getDefaultLocale(_fs: AbstractFileSystem, _rootUri: string): Promise<string> {
+  // In platformOS, en.yml is always the reference translation file
+  return 'en';
+}
+
 async function getDefaultTranslations(
   fs: AbstractFileSystem,
-  theme: Theme,
+  app: App,
   rootUri: string,
-  postfix: string,
 ): Promise<Translations> {
   try {
-    const bufferTranslations = getDefaultTranslationsFromBuffer(theme, postfix);
+    const bufferTranslations = getDefaultTranslationsFromBuffer(app);
     if (bufferTranslations) return bufferTranslations;
-    const defaultLocaleFile = await getDefaultLocaleFile(fs, rootUri, postfix);
+    const defaultLocaleFile = await getDefaultLocaleFile(fs, rootUri);
     if (!defaultLocaleFile) return {};
-    const defaultTranslationsFile = await fs.readFile(defaultLocaleFile);
-    return parseJSON(defaultTranslationsFile, {});
+    const yamlContent = await fs.readFile(defaultLocaleFile);
+    const data = load(yamlContent) as Record<string, any>;
+    if (!data || typeof data !== 'object') return {};
+    // YAML translation files wrap content under the locale key: { en: { hello: 'Hello' } }
+    const localeKey = Object.keys(data)[0];
+    return (localeKey && data[localeKey]) ?? {};
   } catch (error) {
     console.error(error);
     return {};
@@ -105,17 +116,19 @@ async function getDefaultTranslations(
 }
 
 /** It might be that you have an open buffer, we prefer translations from there if available */
-function getDefaultTranslationsFromBuffer(theme: Theme, postfix: string): Translations | undefined {
-  const defaultTranslationsSourceCode = theme.find(
-    (sourceCode) =>
-      sourceCode.type === SourceCodeType.JSON &&
-      sourceCode.uri.match(/locales/) &&
-      sourceCode.uri.endsWith(postfix),
+function getDefaultTranslationsFromBuffer(app: App): Translations | undefined {
+  const defaultTranslationsSourceCode = app.find(
+    (sourceCode) => sourceCode.type === SourceCodeType.YAML && sourceCode.uri.endsWith('/en.yml'),
   );
   if (!defaultTranslationsSourceCode) return undefined;
-  const translations = parseJSON(defaultTranslationsSourceCode.source);
-  if (isError(translations)) return undefined;
-  return translations;
+  try {
+    const data = load(defaultTranslationsSourceCode.source) as Record<string, any>;
+    if (!data || typeof data !== 'object') return undefined;
+    const localeKey = Object.keys(data)[0];
+    return (localeKey && data[localeKey]) ?? undefined;
+  } catch {
+    return undefined;
+  }
 }
 
 function cached<T>(fn: () => Promise<T>): () => Promise<T>;
@@ -157,64 +170,3 @@ const ignoredFolders = ['.git', 'node_modules', 'dist', 'build', 'tmp', 'vendor'
 function isIgnored([uri, type]: FileTuple) {
   return type === FileType.Directory && ignoredFolders.some((folder) => uri.endsWith(folder));
 }
-
-export const FETCHED_METAFIELD_CATEGORIES: MetafieldCategory[] = [
-  'article',
-  'blog',
-  'collection',
-  'company',
-  'company_location',
-  'location',
-  'market',
-  'order',
-  'page',
-  'product',
-  'variant',
-  'shop',
-];
-
-export const makeGetMetafieldDefinitions = (fs: AbstractFileSystem) =>
-  async function (rootUri: string): Promise<MetafieldDefinitionMap> {
-    const definitions = {
-      article: [],
-      blog: [],
-      collection: [],
-      company: [],
-      company_location: [],
-      location: [],
-      market: [],
-      order: [],
-      page: [],
-      product: [],
-      variant: [],
-      shop: [],
-    } as MetafieldDefinitionMap;
-
-    try {
-      const content = await fs.readFile(join(rootUri, '.shopify', 'metafields.json'));
-      const json = parseJSON(content);
-
-      if (isError(json)) return definitions;
-
-      return FETCHED_METAFIELD_CATEGORIES.reduce((definitions, group) => {
-        try {
-          definitions[group] = json[group].map((definition: any) => ({
-            key: definition.key,
-            name: definition.name,
-            namespace: definition.namespace,
-            description: definition.description,
-            type: {
-              category: definition.type.category,
-              name: definition.type.name,
-            },
-          }));
-        } catch (error) {
-          // If there are errors in the file, we ignore it
-        }
-
-        return definitions;
-      }, definitions);
-    } catch (err) {
-      return definitions;
-    }
-  };

package/src/disabled-checks/index.spec.ts

@@ -34,10 +34,10 @@ describe('Module: DisabledChecks', () => {
   describe(`Comment variant`, () => {
     it('should disable checks for the entire document if comment is placed on the first line', async () => {
       for (const buildComment of commentTypes) {
-        const file = `${buildComment('theme-check-disable LiquidFilter')}
+        const file = `${buildComment('platformos-check-disable LiquidFilter')}
 {% comment %}
   This is some comment about the file...
-  Adding a comment here should not mess with theme-check-disable
+  Adding a comment here should not mess with platformos-check-disable
 {% endcomment %}
 {{ 'asset' | random_filter }}
 {% render 'something' %}`;
@@ -50,7 +50,7 @@ describe('Module: DisabledChecks', () => {
 
     it('should disable all checks even if comment has additional spaces', async () => {
       for (const buildComment of commentTypes) {
-        const file = `${buildComment('  theme-check-disable     ')}
+        const file = `${buildComment('  platformos-check-disable     ')}
 {{ 'asset' | random_filter }}
 {% render 'something' %}`;
 
@@ -63,10 +63,10 @@ describe('Module: DisabledChecks', () => {
       for (const buildComment of commentTypes) {
         const file = `{{ 'asset-1' | random_filter }}
 {% render 'something-1' %}
-${buildComment('theme-check-disable')}
+${buildComment('platformos-check-disable')}
 {{ 'asset-2' | random_filter }}
 {% render 'something-2' %}
-${buildComment('theme-check-enable')}
+${buildComment('platformos-check-enable')}
 {{ 'asset-3' | random_filter }}`;
 
         const offenses = await check({ 'code.liquid': file }, checks);
@@ -79,10 +79,10 @@ ${buildComment('theme-check-enable')}
 
     it('should disable a specific check if check is included in the comment', async () => {
       for (const buildComment of commentTypes) {
-        const file = `${buildComment('theme-check-disable LiquidFilter')}
+        const file = `${buildComment('platformos-check-disable LiquidFilter')}
 {{ 'asset-1' | random_filter }}
 {% render 'something' %}
-${buildComment('theme-check-enable LiquidFilter')}
+${buildComment('platformos-check-enable LiquidFilter')}
 {{ 'asset-2' | random_filter }}`;
 
         const offenses = await check({ 'code.liquid': file }, checks);
@@ -94,10 +94,10 @@ ${buildComment('theme-check-enable LiquidFilter')}
 
     it('should disable multiple checks if checks are separated by a comma (and maybe some spaces)', async () => {
       for (const buildComment of commentTypes) {
-        const file = `${buildComment('theme-check-disable LiquidFilter, RenderMarkup')}
+        const file = `${buildComment('platformos-check-disable LiquidFilter, RenderMarkup')}
 {{ 'asset-1' | random_filter }}
 {% render 'something' %}
-${buildComment('theme-check-enable LiquidFilter,RenderMarkup')}
+${buildComment('platformos-check-enable LiquidFilter,RenderMarkup')}
 {{ 'asset-2' | random_filter }}`;
 
         const offenses = await check({ 'code.liquid': file }, checks);
@@ -108,13 +108,13 @@ ${buildComment('theme-check-enable LiquidFilter,RenderMarkup')}
 
     it('should enable specific checks individually', async () => {
       for (const buildComment of commentTypes) {
-        const file = `${buildComment('theme-check-disable LiquidFilter, RenderMarkup')}
+        const file = `${buildComment('platformos-check-disable LiquidFilter, RenderMarkup')}
 {{ 'asset-1' | random_filter }}
 {% render 'something-1' %}
-${buildComment('theme-check-enable RenderMarkup')}
+${buildComment('platformos-check-enable RenderMarkup')}
 {{ 'asset-2' | random_filter }}
 {% render 'something-2' %}
-${buildComment('theme-check-enable LiquidFilter')}
+${buildComment('platformos-check-enable LiquidFilter')}
 {{ 'asset-3' | random_filter }}
 {% render 'something-3' %}`;
 
@@ -129,10 +129,10 @@ ${buildComment('theme-check-enable LiquidFilter')}
     describe('Mix of general and specific commands', () => {
       it('should not reenable specific check when all checks have been disabled before', async () => {
         for (const buildComment of commentTypes) {
-          const file = `${buildComment('theme-check-disable')}
+          const file = `${buildComment('platformos-check-disable')}
 {{ 'asset-1' | random_filter }}
 {% render 'something-1' %}
-${buildComment('theme-check-enable RenderMarkup')}
+${buildComment('platformos-check-enable RenderMarkup')}
 {{ 'asset-2' | random_filter }}
 {% render 'something-2' %}`;
           const offenses = await check({ 'code.liquid': file }, checks);
@@ -142,10 +142,10 @@ ${buildComment('theme-check-enable RenderMarkup')}
 
       it('should reenable all checks when specific ones have been disabled before', async () => {
         for (const buildComment of commentTypes) {
-          const file = `${buildComment('theme-check-disable LiquidFilter, RenderMarkup')}
+          const file = `${buildComment('platformos-check-disable LiquidFilter, RenderMarkup')}
 {{ 'asset-3' | random_filter }}
 {% render 'something-3' %}
-${buildComment('theme-check-enable')}
+${buildComment('platformos-check-enable')}
 {{ 'asset-4' | random_filter }}
 {% render 'something-4' %}`;
 
@@ -159,7 +159,7 @@ ${buildComment('theme-check-enable')}
 
     describe('disable next line', () => {
       it('should disable the next line if there is one', async () => {
-        const file = `{% # theme-check-disable-next-line %}
+        const file = `{% # platformos-check-disable-next-line %}
 {% render 'something' %}
 {% render 'other-thing' %}`;
 
@@ -168,8 +168,8 @@ ${buildComment('theme-check-enable')}
         expectRenderMarkupOffense(offenses, 'other-thing.liquid');
       });
 
-      it("should not disable the liquid tag's children node if theme check is disabled", async () => {
-        const file = `{% # theme-check-disable-next-line %}
+      it("should not disable the liquid tag's children node if platformos-check is disabled", async () => {
+        const file = `{% # platformos-check-disable-next-line %}
 {% if condition %}
   {% render 'something' %}
 {% endif %}
@@ -183,7 +183,7 @@ ${buildComment('theme-check-enable')}
 
       it('should disable the next line inside a liquid tag if there is one', async () => {
         const file = `{% liquid
-  # theme-check-disable-next-line
+  # platformos-check-disable-next-line
   render 'something'
   render 'other-thing'
 %}`;
@@ -193,10 +193,10 @@ ${buildComment('theme-check-enable')}
         expectRenderMarkupOffense(offenses, 'other-thing.liquid');
       });
 
-      it("should disable the parent node's next node if theme check is disabled as the last child node", async () => {
+      it("should disable the parent node's next node if platformos-check is disabled as the last child node", async () => {
         const file = `{% liquid
   if condition
-    # theme-check-disable-next-line
+    # platformos-check-disable-next-line
   elsif other_condition
   endif
 %}`;
@@ -210,11 +210,11 @@ ${buildComment('theme-check-enable')}
         );
       });
 
-      it('should not disable any checks if theme-check is disabled at the end', async () => {
+      it('should not disable any checks if platformos-check is disabled at the end', async () => {
         const file = `{% liquid
   echo hello
   echo everyone
-  # theme-check-disable-next-line
+  # platformos-check-disable-next-line
 %}`;
 
         const offenses = await check({ 'code.liquid': file }, [UndefinedObject]);
@@ -232,7 +232,7 @@ ${buildComment('theme-check-enable')}
       });
 
       it('should disable the next line if the content is an HTML tag with liquid', async () => {
-        const file = `{% # theme-check-disable-next-line %}
+        const file = `{% # platformos-check-disable-next-line %}
 <div class="{{ foo }}"></div>
 <div class="{{ bar }}"></div>`;
 
@@ -246,7 +246,7 @@ ${buildComment('theme-check-enable')}
       });
 
       it('should not disable the next line if the specified rule does not exist', async () => {
-        const file = `{% # theme-check-disable-next-line FAKE_RULE %}
+        const file = `{% # platformos-check-disable-next-line FAKE_RULE %}
 <div class="{{ foo }}"></div>`;
 
         const offenses = await check({ 'code.liquid': file }, [UndefinedObject]);

package/src/disabled-checks/index.ts

@@ -24,7 +24,7 @@ export function createDisabledChecksModule() {
     node: LiquidTag | LiquidRawTag,
   ) {
     const [_, command, checksJoined] =
-      value.trim().match(/^(?:theme\-check\-(disable-next-line|disable|enable)) ?(.*)/) || [];
+      value.trim().match(/^(?:platformos\-check\-(disable-next-line|disable|enable)) ?(.*)/) || [];
 
     const checks = checksJoined ? checksJoined.split(/,[ ]*/) : [SPECIFIC_CHECK_NOT_DEFINED];
 
@@ -133,7 +133,7 @@ export function findNextLinePosition(
    * E.g. The following disables check for `elsif` tag
    *
    * {% if condition %}
-   *   {% #theme-check-disable-next-line %}
+   *   {% # platformos-check-disable-next-line %}
    * {% elsif other_condition %}
    *   {{ prouduct }}
    * {% endif %}

package/src/disabled-checks/test-checks.ts

@@ -45,14 +45,14 @@ export const RenderMarkup: LiquidCheckDefinition = {
   create(context) {
     return {
       async RenderMarkup(node) {
-        if (node.snippet.type === NodeTypes.VariableLookup) {
+        if (node.partial.type === NodeTypes.VariableLookup) {
           return;
         }
 
         context.report({
-          message: `'${node.snippet.value}.liquid' can not be rendered`,
-          startIndex: node.snippet.position.start,
-          endIndex: node.snippet.position.end,
+          message: `'${node.partial.value}.liquid' can not be rendered`,
+          startIndex: node.partial.position.start,
+          endIndex: node.partial.position.end,
         });
       },
     };

package/src/find-root.ts

@@ -5,13 +5,11 @@ type FileExists = (uri: string) => Promise<boolean>;
 
 async function isRoot(dir: UriString, fileExists: FileExists) {
   return or(
-    // .pos config file and app directory exists
-    and(
-      fileExists(path.join(dir, '.pos')),
-      fileExists(path.join(dir, 'app')),
-      fileExists(path.join(dir, 'modules')),
-      fileExists(path.join(dir, '.git')),
-    ),
+    fileExists(path.join(dir, '.pos')),
+    fileExists(path.join(dir, '.platformos-check.yml')),
+    fileExists(path.join(dir, 'app')),
+    // modules/ is a root indicator only when not inside app/ (app/modules/ is a valid subdirectory)
+    and(fileExists(path.join(dir, 'modules')), Promise.resolve(path.basename(dir) !== 'app')),
   );
 }
 
@@ -25,24 +23,16 @@ async function or(...promises: Promise<boolean>[]) {
   return bools.reduce((a, b) => a || b, false);
 }
 
-async function not(ap: Promise<boolean>) {
-  const a = await ap;
-  return !a;
-}
-
 /**
- * Returns the "root" of a theme or theme app extension. The root is the
- * directory that contains a `.theme-check.yml` file, a `.git` directory, or a
- * `shopify.extension.toml` file.
- *
- * There are cases where .theme-check.yml is not defined and we have to infer the root.
- * We'll assume that the root is the directory that contains a `snippets` directory.
+ * Returns the root of a platformOS app. The root is the directory that contains
+ * a `.pos` sentinel file, a `.platformos-check.yml` config file, an `app/` directory,
+ * or a `modules/` directory (when not inside `app/`).
  *
- * So you can think of this function as the function that infers where a .theme-check.yml
- * should be.
+ * Note: `modules/` inside `app/` (i.e. `app/modules/`) is a valid subdirectory and
+ * should not be treated as a root indicator.
  *
- * Note: that this is not the theme root. The config file might have a `root` entry in it
- * that points to somewhere else.
+ * Note: this is not the app root itself. The config file might have a `root` entry that
+ * points to somewhere else.
  */
 export async function findRoot(curr: UriString, fileExists: FileExists): Promise<UriString | null> {
   const currIsRoot = await isRoot(curr, fileExists);

package/src/fixes/autofix.spec.ts

@@ -4,7 +4,7 @@ import { Offense, SourceCodeType } from '../types';
 
 describe('Module: autofix', () => {
   it('should apply a list of all safe changes', async () => {
-    const mockTheme = {
+    const mockApp = {
       'a.liquid': 'Banana world',
       'b.json': prettyJSON({ a: 'b' }),
     };
@@ -69,7 +69,7 @@ describe('Module: autofix', () => {
       },
     ];
 
-    const fixed = await autofix(mockTheme, offenses);
+    const fixed = await autofix(mockApp, offenses);
     expect(fixed['a.liquid']).to.eql('Bananananana world');
     expect(fixed['b.json']).to.eql(
       prettyJSON({

package/src/fixes/autofix.ts

@@ -1,4 +1,4 @@
-import { FixApplicator, Offense, SourceCodeType, Theme } from '../types';
+import { FixApplicator, Offense, SourceCodeType, App } from '../types';
 import { WithRequired } from '../utils/types';
 import { createCorrector } from './correctors';
 import { flattenFixes } from './utils';
@@ -8,13 +8,13 @@ type FixableOffense<S extends SourceCodeType> = S extends SourceCodeType
   : never;
 
 /**
- * Takes a theme, list of offenses and a fixApplicator and runs all the
- * safe ones on the theme.
+ * Takes an app, list of offenses and a fixApplicator and runs all the
+ * safe ones on the app.
  *
  * Note that offense.fix is assumed to be safe, unlike offense.suggest
  * options.
  */
-export async function autofix(sourceCodes: Theme, offenses: Offense[], applyFixes: FixApplicator) {
+export async function autofix(sourceCodes: App, offenses: Offense[], applyFixes: FixApplicator) {
   const fixableOffenses = offenses.filter(
     (offense): offense is FixableOffense<SourceCodeType> => 'fix' in offense && !!offense.fix,
   );

package/src/fixes/correctors/index.ts

@@ -18,6 +18,10 @@ export function createCorrector<S extends SourceCodeType>(
     }
     case SourceCodeType.GraphQL:
       return new GraphQLCorrector(source) as Corrector<typeof sourceCodeType>;
+    case SourceCodeType.YAML: {
+      // YAML autofix is not yet supported; this case should not be reached
+      throw new Error('YAML autofix is not supported');
+    }
     default: {
       return assertNever(sourceCodeType);
     }

package/src/ignore.spec.ts

@@ -47,7 +47,7 @@ describe('Function: isIgnored', () => {
     const result = isIgnored(
       toUri('app/views/partials/foo.liquid'),
       config({
-        checkIgnore: ['!snippets/*'],
+        checkIgnore: ['!other-dir/*'],
         globalIgnore: [],
       }),
       checkDef,
@@ -73,7 +73,7 @@ describe('Function: isIgnored', () => {
     const result = isIgnored(
       toUri('app/views/partials/foo.liquid'),
       config({
-        checkIgnore: ['other-snippets/*.liquid'],
+        checkIgnore: ['other-dir/*.liquid'],
         globalIgnore: [],
       }),
       checkDef,
@@ -165,10 +165,10 @@ describe('Function: isIgnored', () => {
 
   it('should work with only global ignore as well', () => {
     const result = isIgnored(
-      toUri('layout/theme.liquid'),
+      toUri('app/views/layouts/layout.liquid'),
       config({
         checkIgnore: [],
-        globalIgnore: ['layout/theme.liquid'],
+        globalIgnore: ['app/views/layouts/layout.liquid'],
       }),
     );
 
@@ -188,7 +188,6 @@ function config({
   globalIgnore?: string[];
 }): Config {
   return {
-    context: 'theme',
     settings: {
       MockCheck: {
         enabled: true,