oracle-samples / oracle-samples/ofs-proxy-js

Compatibility: support Oracle JET/OJET MVVC release builds (AMD/RequireJS)

Open
#77 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

  • #78 by @miquelgall — closed without merging
Dominant language
TypeScript
Stars
3
Forks
1
PR merge metrics
No merged PRs in 30d

Description

Problem

Using @ofs-users/proxy.js in Oracle OJET MVVC release builds fails.

Observed with ojet build --release (or just build in release mode):

  • Build attempts to resolve AMD path: web/js/@ofs-users/proxy.js
  • Resolution fails with ENOENT

Context

  • Current package is ESM-first ("type": "module", main: dist/ofs.es.js)
  • Sample/plugin build pipeline is RequireJS/AMD-oriented
  • Direct import currently breaks OJET release build flow

Additional Typings Gap

  • Assistants params typings currently expose dateFrom
  • dateTo is not exposed in the same assistants params interface

Requested Outcome

Make the library compatible with Oracle OJET MVVC release builds while preserving backward compatibility for existing plugins already using the library.

Acceptance Criteria

  • ojet build --release succeeds when consuming this library in OJET MVVC context
  • Packaging/import strategy works for AMD/RequireJS consumers (without breaking existing consumers)
  • Existing plugin integrations keep working (backward compatibility)
  • Assistants params typings include dateTo (alongside dateFrom) in a non-breaking way

Notes

Please evaluate options such as dual package output (ESM + AMD/UMD-compatible entry), conditional exports, and/or documented OJET-safe import path with backward compatibility guarantees.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by inspecting the package metadata, especially "type" and "main", the packaging/import setup, and the assistants params interface where dateFrom is exposed. Run ojet build --release in the OJET MVVC context to reproduce the AMD resolution failure. Done means the release build succeeds, existing plugin integrations remain compatible, and dateTo is added non-breakingly.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, typescript
Domain
build-system, developer-experience
Issue type
Feature
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.