fix: skip false embedded scan candidates with optional markers

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 #### Review fixes
 
+- Skipped false embedded-todo scan candidates when custom expressions match without a non-empty marker.
 - Accepted an explicit leading `+` in time estimates and other duration expressions, alongside existing negative durations.
 - Preserved balanced closing parentheses, brackets, and braces in relative file-link filenames while excluding surrounding punctuation.
 - Preserved unsaved archive buffers, serialized simultaneous transfers, and replaced closed archive files only after a complete write. Failed writes, stale source documents, and rejected editor edits keep source tasks intact.

src/utils/embedded/regex.ts

@@ -33,8 +33,9 @@ export const hasEmbeddedMatch = (content: string, regex: RegExp): boolean => {
   const matcher = new RegExp(regex.source, regex.flags);
 
   return splitLines(content).some((line) => {
+    const trimmed = _.trimStart(line);
     matcher.lastIndex = 0;
 
-    return matcher.test(_.trimStart(line));
+    return matcher.test(trimmed) && parseEmbeddedMatches(trimmed, regex).length > 0;
   });
 };

test/embedded-regex.spec.ts

@@ -80,6 +80,18 @@ describe('Embedded todo scan pre-check', () => {
 
     expect(hasEmbeddedMatch('const ready = true;\r  // TASK ship it', regex)).to.equal(true);
   });
+
+  it('skips matches without a configured marker capture', () => {
+    const regex = /(TODO)?(hello)/g;
+    expect(hasEmbeddedMatch('hello', regex)).to.equal(false);
+    expect(hasEmbeddedMatch('TODOhello', regex)).to.equal(true);
+  });
+
+  it('finds a later valid match after an empty alternative', () => {
+    const regex = /(TODO)|(?:)/g;
+    expect(hasEmbeddedMatch('nothing', regex)).to.equal(false);
+    expect(hasEmbeddedMatch(' TODO', regex)).to.equal(true);
+  });
 });
 
 describe('Embedded todo line parsing', () => {