Skip to content

finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887

Closed
#7013
Closed
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
#7013
Assignees
os-sam
Labels
domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3

Description

@os-sam
os-sam
opened on Aug 30, 2026

Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).

The observation

packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:

interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}

It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.

What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:

This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.

Why it is only a finding and not a bug

Measured, so the grading is not a guess:

  • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
  • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
  • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.

So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.

Candidate directions (not a recommendation — this needs triage, not a fix from me)

  1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
  2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
  3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".

⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.

Metadata

Metadata

Assignees

Labels

domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    , 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
     blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
    Skip to content
    finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
    Closed
    #7013
    Closed
    finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
    #7013
    Assignees
    os-sam
    Labels
    domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
    Description
    @os-sam
    os-sam
    opened on Aug 30, 2026
    Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
    The observation
    packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
    interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
    It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
    What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
    This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
    Why it is only a finding and not a bug
    Measured, so the grading is not a guess:
    • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
    • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
    • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
    So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
    Candidate directions (not a recommendation — this needs triage, not a fix from me)
    1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
    2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
    3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
    ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
    Metadata
    Metadata
    Assignees
    Labels
    domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
    Type
    No type
    Projects
    No projects
      Milestone
      No milestone
      Relationships
      None yet
      Development
      No branches or pull requests
      Issue actions
      
, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
      Skip to content
      finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
      Closed
      #7013
      Closed
      finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
      #7013
      Assignees
      os-sam
      Labels
      domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
      Description
      @os-sam
      os-sam
      opened on Aug 30, 2026
      Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
      The observation
      packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
      interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
      It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
      What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
      This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
      Why it is only a finding and not a bug
      Measured, so the grading is not a guess:
      • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
      • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
      • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
      So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
      Candidate directions (not a recommendation — this needs triage, not a fix from me)
      1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
      2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
      3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
      ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
      Metadata
      Metadata
      Assignees
      Labels
      domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
      Type
      No type
      Projects
      No projects
        Milestone
        No milestone
        Relationships
        None yet
        Development
        No branches or pull requests
        Issue actions
        
, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
        Skip to content
        finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
        Closed
        #7013
        Closed
        finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
        #7013
        Assignees
        os-sam
        Labels
        domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
        Description
        @os-sam
        os-sam
        opened on Aug 30, 2026
        Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
        The observation
        packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
        interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
        It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
        What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
        This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
        Why it is only a finding and not a bug
        Measured, so the grading is not a guess:
        • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
        • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
        • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
        So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
        Candidate directions (not a recommendation — this needs triage, not a fix from me)
        1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
        2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
        3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
        ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
        Metadata
        Metadata
        Assignees
        Labels
        domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
        Type
        No type
        Projects
        No projects
          Milestone
          No milestone
          Relationships
          None yet
          Development
          No branches or pull requests
          Issue actions
          
, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
          Skip to content
          finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
          Closed
          #7013
          Closed
          finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
          #7013
          Assignees
          os-sam
          Labels
          domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
          Description
          @os-sam
          os-sam
          opened on Aug 30, 2026
          Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
          The observation
          packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
          interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
          It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
          What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
          This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
          Why it is only a finding and not a bug
          Measured, so the grading is not a guess:
          • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
          • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
          • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
          So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
          Candidate directions (not a recommendation — this needs triage, not a fix from me)
          1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
          2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
          3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
          ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
          Metadata
          Metadata
          Assignees
          Labels
          domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
          Type
          No type
          Projects
          No projects
            Milestone
            No milestone
            Relationships
            None yet
            Development
            No branches or pull requests
            Issue actions
            
, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
            Skip to content
            finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
            Closed
            #7013
            Closed
            finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
            #7013
            Assignees
            os-sam
            Labels
            domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
            Description
            @os-sam
            os-sam
            opened on Aug 30, 2026
            Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
            The observation
            packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
            interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
            It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
            What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
            This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
            Why it is only a finding and not a bug
            Measured, so the grading is not a guess:
            • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
            • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
            • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
            So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
            Candidate directions (not a recommendation — this needs triage, not a fix from me)
            1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
            2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
            3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
            ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
            Metadata
            Metadata
            Assignees
            Labels
            domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
            Type
            No type
            Projects
            No projects
              Milestone
              No milestone
              Relationships
              None yet
              Development
              No branches or pull requests
              Issue actions
              
, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
              Skip to content
              finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
              Closed
              #7013
              Closed
              finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
              #7013
              Assignees
              os-sam
              Labels
              domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
              Description
              @os-sam
              os-sam
              opened on Aug 30, 2026
              Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
              The observation
              packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
              interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
              It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
              What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
              This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
              Why it is only a finding and not a bug
              Measured, so the grading is not a guess:
              • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
              • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
              • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
              So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
              Candidate directions (not a recommendation — this needs triage, not a fix from me)
              1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
              2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
              3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
              ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
              Metadata
              Metadata
              Assignees
              Labels
              domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
              Type
              No type
              Projects
              No projects
                Milestone
                No milestone
                Relationships
                None yet
                Development
                No branches or pull requests
                Issue actions
                
, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it · Issue #6887 · objectstack-ai/objectui · GitHub
                Skip to content
                finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it #6887
                Closed
                #7013
                Closed
                finding(components): metadata-viewer hand-writes a fifth copy of the select-option shape instead of deriving it#6887
                #7013
                Assignees
                os-sam
                Labels
                domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
                Description
                @os-sam
                os-sam
                opened on Aug 30, 2026
                Filing unassigned — recording, not claiming. Found while re-verifying #6263 (per-option default on the form-view face); this is a different surface and is out of scope for that card's PR (#6886).
                The observation
                packages/components/src/renderers/basic/metadata-viewer.tsx:109 declares its own local shape for a select option:
                interfaceSelectOption{label?: string;value: string;color?: string;default?: boolean;}
                It is used at line 144 to read an object definition'sfield.options off the (untyped) metadata context, and at line 148 to pick a state machine's initial state via options.find((o) =greater-than o.default). That reader is correct and the key it reads is the ruled-enforce object-field face (objectstack#7246 / PR #7388) — ⛔ nothing here suggests changing the behaviour.
                What is worth recording is the shape declaration, not the behaviour. This repo has been closing exactly this defect class for several cards running — one contract, several hand-written descriptions, each only ever checked against itself:
                This declaration is in the same class and was not part of any of those sweeps: four of the spec's five SelectOption keys, visibleWhen absent, no Omit naming the narrowing, no comment saying it is deliberate.
                Why it is only a finding and not a bug
                Measured, so the grading is not a guess:
                • It is a read model, not an authoring surface. Nothing type-checks author-written metadata against it — the value arrives from useMetadataItem as any — so unlike A per-option visibleWhen written in a *.form.ts is silently inert — the metadata-admin renderer never reads it #6247 there is no author who can be told "this parsed clean" and then be ignored.
                • The renderer's behaviour is right today: it reads default off an object definition, which is where the key is enforced.
                • scripts/check-spec-symbol-derivation.mjs reports 2 declared deliberate copies, 18 unbacked claims in 5 packages and this declaration is not among the flagged claims — it carries no alignment claim comment at all, so the gate has nothing to match on. That is the gap: the gate can only see copies that say they are copies.
                So the cost is drift risk, not a live defect: if the spec's option shape moves, this copy goes on compiling and silently answers an older question.
                Candidate directions (not a recommendation — this needs triage, not a fix from me)
                1. Derive it — Omit-with-named-narrowings from @objectstack/spec/data's SelectOption, the sanctioned form in this repo, which is what form-spec.ts now does one package over.
                2. Leave it and add the alignment-claim comment the derivation gate looks for, so it becomes a declared deliberate copy rather than an invisible one.
                3. Ask whether the gate should notice un-claimed structural copies at all — the class it currently catches is "claims alignment but is not backed", and this one is the neighbouring class, "is a copy and says nothing".
                ⛔ Out of scope for anyone picking this up: the object-field face's per-option default is ruled enforce (objectstack#7246, maintainer override recorded verbatim as "enforce.", implemented by PR #7388), and the alias table (isDefault / selected) is likewise untouchable. This card is about where a shape is declared, not about what any surface honours.
                Metadata
                Metadata
                Assignees
                Labels
                domain:uiobjectui ui stream: fix lands on the published library or apps — objectui execution seatpackage: componentspm:dispatchedpriority:p3
                Type
                No type
                Projects
                No projects
                  Milestone
                  No milestone
                  Relationships
                  None yet
                  Development
                  No branches or pull requests
                  Issue actions