feat: added config proxy

[RohitKushvaha01]

No description provided.

[greptile-apps]

Greptile Summary

This PR hardens the plugin-facing config API by wrapping the config object in a Proxy before exposing it through acode.require("config"). Internal code continues to import config directly and retains full access (including the HAS_PRO setter), while plugin code receives a read-only view that silently blocks all mutation traps.

• The Proxy correctly returns true from all mutation traps (set, defineProperty, deleteProperty, setPrototypeOf), which is the right behaviour for silent rejection in strict-mode ES modules.
• SKU_LIST is now Object.freeze()'d at the source, closing the by-reference mutation loophole for that array.
• No get trap is present, so accessor properties like HAS_PRO and PLAY_STORE_URL continue to be forwarded to the underlying getters correctly.

Confidence Score: 5/5

Safe to merge — the proxy correctly silences all mutation traps with return true, internal Acode code retains direct mutable access to config, and the SKU_LIST freeze closes the only by-reference array loophole.

All mutation traps return true (correct for strict-mode modules), no get trap means accessor properties like HAS_PRO and PLAY_STORE_URL forward to their underlying getters without issue, and the Object.freeze on SKU_LIST addresses the one concrete mutable nested value. Previously raised issues (false-returning traps, redundant get trap) are not present in this version of the code.

No files require special attention.

Important Files Changed

Filename	Overview
src/lib/acode.js	Wraps the exported config object in a Proxy that silently blocks all mutation operations (set, defineProperty, deleteProperty, setPrototypeOf) before registering it with this.define("config", configProxy) — guards plugin-facing access while leaving internal direct imports of config unaffected.
src/lib/config.js	Converts SKU_LIST from a plain array to an Object.freeze()'d array, preventing in-place mutation of that array even when accessed by reference through the proxy.

Sequence Diagram

%%{init: {'theme': 'neutral'}}%%
sequenceDiagram
    participant Plugin
    participant ConfigProxy as Proxy(config)
    participant Config as config (config.js)

    Note over Plugin,Config: Plugin access via acode.require("config")

    Plugin->>ConfigProxy: read prop (e.g. SKU_LIST, HAS_PRO)
    ConfigProxy->>Config: Reflect.get(config, prop, receiver)
    Config-->>Plugin: value (frozen array / getter result)

    Plugin->>ConfigProxy: "write prop (e.g. HAS_PRO = true)"
    ConfigProxy-->>Plugin: console.warn + return true (silent block)

    Plugin->>ConfigProxy: delete prop
    ConfigProxy-->>Plugin: console.warn + return true (silent block)

    Note over Config: Internal Acode code imports config directly

    Config->>Config: "config.HAS_PRO = true (unblocked)"
    Config->>Config: config.SKU_LIST[0] (frozen array)

Loading

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
sequenceDiagram
    participant Plugin
    participant ConfigProxy as Proxy(config)
    participant Config as config (config.js)

    Note over Plugin,Config: Plugin access via acode.require("config")

    Plugin->>ConfigProxy: read prop (e.g. SKU_LIST, HAS_PRO)
    ConfigProxy->>Config: Reflect.get(config, prop, receiver)
    Config-->>Plugin: value (frozen array / getter result)

    Plugin->>ConfigProxy: "write prop (e.g. HAS_PRO = true)"
    ConfigProxy-->>Plugin: console.warn + return true (silent block)

    Plugin->>ConfigProxy: delete prop
    ConfigProxy-->>Plugin: console.warn + return true (silent block)

    Note over Config: Internal Acode code imports config directly

    Config->>Config: "config.HAS_PRO = true (unblocked)"
    Config->>Config: config.SKU_LIST[0] (frozen array)

Loading

_{Reviews (4): Last reviewed commit: "fix: resolved concerns" | Re-trigger Greptile}

[RohitKushvaha01]

@greptileai

[RohitKushvaha01]

@greptileai