Skip to content

Conversation

@sandersn
Copy link
Contributor

The types that webxr shims are now the precise literals instead of number.

The new code won't succeed until tomorrow. See microsoft/TypeScript#52328

The types that webxr shims are now the precise literals instead of
`number`.

The new code won't succeed until tomorrow. See microsoft/TypeScript#52328
@typescript-bot
Copy link
Contributor

typescript-bot commented Jan 21, 2023

@sandersn Thank you for submitting this PR!

This is a live comment which I will keep updated.

7 packages in this PR

Code Reviews

Because this PR edits multiple packages, it can be merged once it's reviewed by a DT maintainer.

You can test the changes of this PR in the Playground.

Status

  • ✅ No merge conflicts
  • ✅ Continuous integration tests have passed
  • 🕐 A DT maintainer needs to approve changes which affect more than one package

Once every item on this list is checked, I'll ask you for permission to merge and publish the changes.


Diagnostic Information: What the bot saw about this PR
{
  "type": "info",
  "now": "-",
  "pr_number": 64012,
  "author": "sandersn",
  "headCommitOid": "c2d52cf06c36b877a232b7d6ae64a970d443f27a",
  "mergeBaseOid": "dd38d16a145c2089fd9da7df263fb8d16e9709c7",
  "lastPushDate": "2023-01-23T17:53:24.000Z",
  "lastActivityDate": "2023-01-23T17:58:07.000Z",
  "hasMergeConflict": false,
  "isFirstContribution": false,
  "tooManyFiles": false,
  "hugeChange": false,
  "popularityLevel": "Popular",
  "pkgInfo": [
    {
      "name": "aframe",
      "kind": "edit",
      "files": [
        {
          "path": "types/aframe/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "devpaul",
        "bertoritger",
        "twastvedt"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "drag-controls",
      "kind": "edit",
      "files": [
        {
          "path": "types/drag-controls/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "MGHawes"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "three-dat.gui",
      "kind": "edit",
      "files": [
        {
          "path": "types/three-dat.gui/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "hkleungai"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "three-gif-loader",
      "kind": "edit",
      "files": [
        {
          "path": "types/three-gif-loader/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "lairjr"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "three-tds-loader",
      "kind": "edit",
      "files": [
        {
          "path": "types/three-tds-loader/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "KonstantinLukaschenko",
        "sschoensee"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Well-liked by everyone"
    },
    {
      "name": "three",
      "kind": "edit",
      "files": [
        {
          "path": "types/three/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "joshuaellis",
        "Methuselah96"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Popular"
    },
    {
      "name": "webxr",
      "kind": "edit",
      "files": [
        {
          "path": "types/webxr/index.d.ts",
          "kind": "definition"
        }
      ],
      "owners": [
        "robrohan",
        "RaananW",
        "capnmidnight"
      ],
      "addedOwners": [],
      "deletedOwners": [],
      "popularityLevel": "Popular"
    }
  ],
  "reviews": [
    {
      "type": "approved",
      "reviewer": "Methuselah96",
      "date": "2023-01-23T17:58:07.000Z",
      "isMaintainer": false
    }
  ],
  "mainBotCommentID": 1399096967,
  "ciResult": "pass"
}

@typescript-bot typescript-bot added Popular package This PR affects a popular package (as counted by NPM download counts). Untested Change This PR does not touch tests labels Jan 21, 2023
@typescript-bot
Copy link
Contributor

typescript-bot commented Jan 21, 2023

🔔 @devpaul @bertoritger @twastvedt @MGHawes @hkleungai @lairjr @KonstantinLukaschenko @sschoensee @joshuaellis @Methuselah96 @robrohan @RaananW @capnmidnight — please review this PR in the next few days. Be sure to explicitly select Approve or Request Changes in the GitHub UI so I know what's going on.

@typescript-bot typescript-bot added the The CI failed When GH Actions fails label Jan 21, 2023
@typescript-bot
Copy link
Contributor

@sandersn The CI build failed! Please review the logs for more information.

Once you've pushed the fixes, the build will automatically re-run. Thanks!

Note: builds which are failing do not end up on the list of PRs for the DT maintainers to review.

@sandersn sandersn closed this Jan 23, 2023
@sandersn sandersn reopened this Jan 23, 2023
@typescript-bot typescript-bot added Edits multiple packages The CI failed When GH Actions fails and removed The CI failed When GH Actions fails labels Jan 23, 2023
@typescript-bot
Copy link
Contributor

@sandersn The CI build failed! Please review the logs for more information.

Once you've pushed the fixes, the build will automatically re-run. Thanks!

Note: builds which are failing do not end up on the list of PRs for the DT maintainers to review.

@typescript-bot typescript-bot removed the The CI failed When GH Actions fails label Jan 23, 2023
@Methuselah96
Copy link
Contributor

I assume that there's no way to support <5.0 and >=5.0 at the same time without separate types?

@DangerBotOSS
Copy link

DangerBotOSS commented Jan 23, 2023

Inspecting the JavaScript source for this package found some properties that are not in the .d.ts files.
The check for missing properties isn't always right, so take this list as advice, not a requirement.

aframe (unpkg)

was missing the following properties:

  1. base64
  2. isMobile
  3. isIOS
  4. isIFrame
  5. appendQueryParameter
as well as these 1233 other properties...

getQueryParameter, isLandscapeMode, byteLength, toByteArray, fromByteArray, Buffer, SlowBuffer, INSPECT_MAX_BYTES, kMaxLength, log, formatArgs, save, load, useColors, storage, colors, coerce, disable, enable, enabled, names, skips, formatters, read, write, wrap, isGeneratorFunction, mark, awrap, AsyncIterator, async, keys, values, ACESFilmicToneMapping, AddEquation, AddOperation, AdditiveAnimationBlendMode, AdditiveBlending, AlphaFormat, AlwaysDepth, AlwaysStencilFunc, AmbientLight, AmbientLightProbe, AnimationClip, AnimationLoader, AnimationMixer, AnimationObjectGroup, AnimationUtils, ArcCurve, ArrayCamera, ArrowHelper, Audio, AudioAnalyser, AudioContext, AudioListener, AudioLoader, AxesHelper, AxisHelper, BackSide, BasicDepthPacking, BasicShadowMap, BinaryTextureLoader, Bone, BooleanKeyframeTrack, BoundingBoxHelper, Box2, Box3, Box3Helper, BoxBufferGeometry, BoxGeometry, BoxHelper, BufferAttribute, BufferGeometry, BufferGeometryLoader, ByteType, Cache, Camera, CameraHelper, CanvasRenderer, CanvasTexture, CatmullRomCurve3, CineonToneMapping, CircleBufferGeometry, CircleGeometry, ClampToEdgeWrapping, Clock, ClosedSplineCurve3, Color, ColorKeyframeTrack, CompressedTexture, CompressedTextureLoader, ConeBufferGeometry, ConeGeometry, CubeCamera, CubeReflectionMapping, CubeRefractionMapping, CubeTexture, CubeTextureLoader, CubeUVReflectionMapping, CubeUVRefractionMapping, CubicBezierCurve, CubicBezierCurve3, CubicInterpolant, CullFaceBack, CullFaceFront, CullFaceFrontBack, CullFaceNone, Curve, CurvePath, CustomBlending, CustomToneMapping, CylinderBufferGeometry, CylinderGeometry, Cylindrical, DataTexture, DataTexture2DArray, DataTexture3D, DataTextureLoader, DataUtils, DecrementStencilOp, DecrementWrapStencilOp, DefaultLoadingManager, DepthFormat, DepthStencilFormat, DepthTexture, DirectionalLight, DirectionalLightHelper, DiscreteInterpolant, DodecahedronBufferGeometry, DodecahedronGeometry, DoubleSide, DstAlphaFactor, DstColorFactor, DynamicBufferAttribute, DynamicCopyUsage, DynamicDrawUsage, DynamicReadUsage, EdgesGeometry, EdgesHelper, EllipseCurve, EqualDepth, EqualStencilFunc, EquirectangularReflectionMapping, EquirectangularRefractionMapping, Euler, EventDispatcher, ExtrudeBufferGeometry, ExtrudeGeometry, Face3, Face4, FaceColors, FileLoader, FlatShading, Float16BufferAttribute, Float32Attribute, Float32BufferAttribute, Float64Attribute, Float64BufferAttribute, FloatType, Fog, FogExp2, Font, FontLoader, FrontSide, Frustum, GLBufferAttribute, GLSL1, GLSL3, GammaEncoding, GeometryUtils, GreaterDepth, GreaterEqualDepth, GreaterEqualStencilFunc, GreaterStencilFunc, GridHelper, Group, HalfFloatType, HemisphereLight, HemisphereLightHelper, HemisphereLightProbe, IcosahedronBufferGeometry, IcosahedronGeometry, ImageBitmapLoader, ImageLoader, ImageUtils, ImmediateRenderObject, IncrementStencilOp, IncrementWrapStencilOp, InstancedBufferAttribute, InstancedBufferGeometry, InstancedInterleavedBuffer, InstancedMesh, Int16Attribute, Int16BufferAttribute, Int32Attribute, Int32BufferAttribute, Int8Attribute, Int8BufferAttribute, IntType, InterleavedBuffer, InterleavedBufferAttribute, Interpolant, InterpolateDiscrete, InterpolateLinear, InterpolateSmooth, InvertStencilOp, JSONLoader, KeepStencilOp, KeyframeTrack, LOD, LatheBufferGeometry, LatheGeometry, Layers, LensFlare, LessDepth, LessEqualDepth, LessEqualStencilFunc, LessStencilFunc, Light, LightProbe, Line, Line3, LineBasicMaterial, LineCurve, LineCurve3, LineDashedMaterial, LineLoop, LinePieces, LineSegments, LineStrip, LinearEncoding, LinearFilter, LinearInterpolant, LinearMipMapLinearFilter, LinearMipMapNearestFilter, LinearMipmapLinearFilter, LinearMipmapNearestFilter, LinearToneMapping, Loader, LoaderUtils, LoadingManager, LogLuvEncoding, LoopOnce, LoopPingPong, LoopRepeat, LuminanceAlphaFormat, LuminanceFormat, MOUSE, Material, MaterialLoader, Math, MathUtils, Matrix3, Matrix4, MaxEquation, Mesh, MeshBasicMaterial, MeshDepthMaterial, MeshDistanceMaterial, MeshFaceMaterial, MeshLambertMaterial, MeshMatcapMaterial, MeshNormalMaterial, MeshPhongMaterial, MeshPhysicalMaterial, MeshStandardMaterial, MeshToonMaterial, MinEquation, MirroredRepeatWrapping, MixOperation, MultiMaterial, MultiplyBlending, MultiplyOperation, NearestFilter, NearestMipMapLinearFilter, NearestMipMapNearestFilter, NearestMipmapLinearFilter, NearestMipmapNearestFilter, NeverDepth, NeverStencilFunc, NoBlending, NoColors, NoToneMapping, NormalAnimationBlendMode, NormalBlending, NotEqualDepth, NotEqualStencilFunc, NumberKeyframeTrack, Object3D, ObjectLoader, ObjectSpaceNormalMap, OctahedronBufferGeometry, OctahedronGeometry, OneFactor, OneMinusDstAlphaFactor, OneMinusDstColorFactor, OneMinusSrcAlphaFactor, OneMinusSrcColorFactor, OrthographicCamera, PCFShadowMap, PCFSoftShadowMap, PMREMGenerator, ParametricBufferGeometry, ParametricGeometry, Particle, ParticleBasicMaterial, ParticleSystem, ParticleSystemMaterial, Path, PerspectiveCamera, Plane, PlaneBufferGeometry, PlaneGeometry, PlaneHelper, PointCloud, PointCloudMaterial, PointLight, PointLightHelper, Points, PointsMaterial, PolarGridHelper, PolyhedronBufferGeometry, PolyhedronGeometry, PositionalAudio, PropertyBinding, PropertyMixer, QuadraticBezierCurve, QuadraticBezierCurve3, Quaternion, QuaternionKeyframeTrack, QuaternionLinearInterpolant, REVISION, RGBADepthPacking, RGBAFormat, RGBAIntegerFormat, RGBA_ASTC_10x10_Format, RGBA_ASTC_10x5_Format, RGBA_ASTC_10x6_Format, RGBA_ASTC_10x8_Format, RGBA_ASTC_12x10_Format, RGBA_ASTC_12x12_Format, RGBA_ASTC_4x4_Format, RGBA_ASTC_5x4_Format, RGBA_ASTC_5x5_Format, RGBA_ASTC_6x5_Format, RGBA_ASTC_6x6_Format, RGBA_ASTC_8x5_Format, RGBA_ASTC_8x6_Format, RGBA_ASTC_8x8_Format, RGBA_BPTC_Format, RGBA_ETC2_EAC_Format, RGBA_PVRTC_2BPPV1_Format, RGBA_PVRTC_4BPPV1_Format, RGBA_S3TC_DXT1_Format, RGBA_S3TC_DXT3_Format, RGBA_S3TC_DXT5_Format, RGBDEncoding, RGBEEncoding, RGBEFormat, RGBFormat, RGBIntegerFormat, RGBM16Encoding, RGBM7Encoding, RGB_ETC1_Format, RGB_ETC2_Format, RGB_PVRTC_2BPPV1_Format, RGB_PVRTC_4BPPV1_Format, RGB_S3TC_DXT1_Format, RGFormat, RGIntegerFormat, RawShaderMaterial, Ray, Raycaster, RectAreaLight, RedFormat, RedIntegerFormat, ReinhardToneMapping, RepeatWrapping, ReplaceStencilOp, ReverseSubtractEquation, RingBufferGeometry, RingGeometry, SRGB8_ALPHA8_ASTC_10x10_Format, SRGB8_ALPHA8_ASTC_10x5_Format, SRGB8_ALPHA8_ASTC_10x6_Format, SRGB8_ALPHA8_ASTC_10x8_Format, SRGB8_ALPHA8_ASTC_12x10_Format, SRGB8_ALPHA8_ASTC_12x12_Format, SRGB8_ALPHA8_ASTC_4x4_Format, SRGB8_ALPHA8_ASTC_5x4_Format, SRGB8_ALPHA8_ASTC_5x5_Format, SRGB8_ALPHA8_ASTC_6x5_Format, SRGB8_ALPHA8_ASTC_6x6_Format, SRGB8_ALPHA8_ASTC_8x5_Format, SRGB8_ALPHA8_ASTC_8x6_Format, SRGB8_ALPHA8_ASTC_8x8_Format, Scene, SceneUtils, ShaderChunk, ShaderLib, ShaderMaterial, ShadowMaterial, Shape, ShapeBufferGeometry, ShapeGeometry, ShapePath, ShapeUtils, ShortType, Skeleton, SkeletonHelper, SkinnedMesh, SmoothShading, Sphere, SphereBufferGeometry, SphereGeometry, Spherical, SphericalHarmonics3, Spline, SplineCurve, SplineCurve3, SpotLight, SpotLightHelper, Sprite, SpriteMaterial, SrcAlphaFactor, SrcAlphaSaturateFactor, SrcColorFactor, StaticCopyUsage, StaticDrawUsage, StaticReadUsage, StereoCamera, StreamCopyUsage, StreamDrawUsage, StreamReadUsage, StringKeyframeTrack, SubtractEquation, SubtractiveBlending, TOUCH, TangentSpaceNormalMap, TetrahedronBufferGeometry, TetrahedronGeometry, TextBufferGeometry, TextGeometry, Texture, TextureLoader, TorusBufferGeometry, TorusGeometry, TorusKnotBufferGeometry, TorusKnotGeometry, Triangle, TriangleFanDrawMode, TriangleStripDrawMode, TrianglesDrawMode, TubeBufferGeometry, TubeGeometry, UVMapping, Uint16Attribute, Uint16BufferAttribute, Uint32Attribute, Uint32BufferAttribute, Uint8Attribute, Uint8BufferAttribute, Uint8ClampedAttribute, Uint8ClampedBufferAttribute, Uniform, UniformsLib, UniformsUtils, UnsignedByteType, UnsignedInt248Type, UnsignedIntType, UnsignedShort4444Type, UnsignedShort5551Type, UnsignedShort565Type, UnsignedShortType, VSMShadowMap, Vector2, Vector3, Vector4, VectorKeyframeTrack, Vertex, VertexColors, VideoTexture, WebGL1Renderer, WebGLCubeRenderTarget, WebGLMultisampleRenderTarget, WebGLRenderTarget, WebGLRenderTargetCube, WebGLRenderer, WebGLUtils, WireframeGeometry, WireframeHelper, WrapAroundEnding, XHRLoader, ZeroCurvatureEnding, ZeroFactor, ZeroSlopeEnding, ZeroStencilOp, sRGBEncoding, computeBox, computeSphere, pages, uvs, positions, attr, index, setTimeout, setInterval, clearTimeout, clearInterval, enroll, unenroll, active, setImmediate, clearImmediate, lines, Component, FONTS, inferResponseType, getFileNameFromURL, knownTags, isNode, wrapMethods, registrationOrderWarnings, geometryNames, Geometry, propertyTypes, registerPropertyType, isValidDefaultValue, isValidDefaultCoordinate, setupCanvas, setup, remove, MOBILE_HEAD_TAGS, inject, isSingleProperty, process, processPropertyDefinition, parseProperties, parseProperty, stringifyProperties, stringifyProperty, shaderNames, Shader, System, definePrimitive, regex, parse, stringify, isCoordinates, isCoordinate, toVector3, isWebXRAvailable, getVRDisplay, checkHeadsetConnected, checkARSupport, isTablet, isMobileDeviceRequestingDesktopSite, isOculusBrowser, isFirefoxReality, isMobileVR, isR7, isLandscape, isBrowserEnvironment, isNodeEnvironment, getComponentPropertyPath, propertyPathCache, getComponentProperty, setComponentProperty, bind, coordinates, debug, device, entity, forceCanvasResizeSafariMobile, isIE11, material, objectPool, split, styleParser, trackedControls, isGearVR, isOculusGo, throttle, throttleTick, debounce, extend, extendDeep, clone, deepEqual, diff, shouldCaptureKeyEvent, splitString, getElData, getUrlParameter, isIframed, findAllScenes, srcLoader, updateMapMaterialFromData, updateMap, updateDistortionMap, handleTextureEvents, isHLS, createPool, clearObject, removeUnusedKeys, toCamelCase, transformKeysToCamelCase, checkControllerPresentAndSetup, isControllerPresentWebVR, isControllerPresentWebXR, findMatchingControllerWebVR, findMatchingControllerWebXR, emitIfAxesChanged, onButtonEvent, base64, isMobile, isIOS, isIFrame, appendQueryParameter, getQueryParameter, isLandscapeMode, byteLength, toByteArray, fromByteArray, Buffer, SlowBuffer, INSPECT_MAX_BYTES, kMaxLength, log, formatArgs, save, load, useColors, storage, colors, coerce, disable, enable, enabled, names, skips, formatters, read, write, wrap, isGeneratorFunction, mark, awrap, AsyncIterator, async, keys, values, ACESFilmicToneMapping, AddEquation, AddOperation, AdditiveAnimationBlendMode, AdditiveBlending, AlphaFormat, AlwaysDepth, AlwaysStencilFunc, AmbientLight, AmbientLightProbe, AnimationClip, AnimationLoader, AnimationMixer, AnimationObjectGroup, AnimationUtils, ArcCurve, ArrayCamera, ArrowHelper, Audio, AudioAnalyser, AudioContext, AudioListener, AudioLoader, AxesHelper, AxisHelper, BackSide, BasicDepthPacking, BasicShadowMap, BinaryTextureLoader, Bone, BooleanKeyframeTrack, BoundingBoxHelper, Box2, Box3, Box3Helper, BoxBufferGeometry, BoxGeometry, BoxHelper, BufferAttribute, BufferGeometry, BufferGeometryLoader, ByteType, Cache, Camera, CameraHelper, CanvasRenderer, CanvasTexture, CatmullRomCurve3, CineonToneMapping, CircleBufferGeometry, CircleGeometry, ClampToEdgeWrapping, Clock, ClosedSplineCurve3, Color, ColorKeyframeTrack, CompressedTexture, CompressedTextureLoader, ConeBufferGeometry, ConeGeometry, CubeCamera, CubeReflectionMapping, CubeRefractionMapping, CubeTexture, CubeTextureLoader, CubeUVReflectionMapping, CubeUVRefractionMapping, CubicBezierCurve, CubicBezierCurve3, CubicInterpolant, CullFaceBack, CullFaceFront, CullFaceFrontBack, CullFaceNone, Curve, CurvePath, CustomBlending, CustomToneMapping, CylinderBufferGeometry, CylinderGeometry, Cylindrical, DataTexture, DataTexture2DArray, DataTexture3D, DataTextureLoader, DataUtils, DecrementStencilOp, DecrementWrapStencilOp, DefaultLoadingManager, DepthFormat, DepthStencilFormat, DepthTexture, DirectionalLight, DirectionalLightHelper, DiscreteInterpolant, DodecahedronBufferGeometry, DodecahedronGeometry, DoubleSide, DstAlphaFactor, DstColorFactor, DynamicBufferAttribute, DynamicCopyUsage, DynamicDrawUsage, DynamicReadUsage, EdgesGeometry, EdgesHelper, EllipseCurve, EqualDepth, EqualStencilFunc, EquirectangularReflectionMapping, EquirectangularRefractionMapping, Euler, EventDispatcher, ExtrudeBufferGeometry, ExtrudeGeometry, Face3, Face4, FaceColors, FileLoader, FlatShading, Float16BufferAttribute, Float32Attribute, Float32BufferAttribute, Float64Attribute, Float64BufferAttribute, FloatType, Fog, FogExp2, Font, FontLoader, FrontSide, Frustum, GLBufferAttribute, GLSL1, GLSL3, GammaEncoding, GeometryUtils, GreaterDepth, GreaterEqualDepth, GreaterEqualStencilFunc, GreaterStencilFunc, GridHelper, Group, HalfFloatType, HemisphereLight, HemisphereLightHelper, HemisphereLightProbe, IcosahedronBufferGeometry, IcosahedronGeometry, ImageBitmapLoader, ImageLoader, ImageUtils, ImmediateRenderObject, IncrementStencilOp, IncrementWrapStencilOp, InstancedBufferAttribute, InstancedBufferGeometry, InstancedInterleavedBuffer, InstancedMesh, Int16Attribute, Int16BufferAttribute, Int32Attribute, Int32BufferAttribute, Int8Attribute, Int8BufferAttribute, IntType, InterleavedBuffer, InterleavedBufferAttribute, Interpolant, InterpolateDiscrete, InterpolateLinear, InterpolateSmooth, InvertStencilOp, JSONLoader, KeepStencilOp, KeyframeTrack, LOD, LatheBufferGeometry, LatheGeometry, Layers, LensFlare, LessDepth, LessEqualDepth, LessEqualStencilFunc, LessStencilFunc, Light, LightProbe, Line, Line3, LineBasicMaterial, LineCurve, LineCurve3, LineDashedMaterial, LineLoop, LinePieces, LineSegments, LineStrip, LinearEncoding, LinearFilter, LinearInterpolant, LinearMipMapLinearFilter, LinearMipMapNearestFilter, LinearMipmapLinearFilter, LinearMipmapNearestFilter, LinearToneMapping, Loader, LoaderUtils, LoadingManager, LogLuvEncoding, LoopOnce, LoopPingPong, LoopRepeat, LuminanceAlphaFormat, LuminanceFormat, MOUSE, Material, MaterialLoader, Math, MathUtils, Matrix3, Matrix4, MaxEquation, Mesh, MeshBasicMaterial, MeshDepthMaterial, MeshDistanceMaterial, MeshFaceMaterial, MeshLambertMaterial, MeshMatcapMaterial, MeshNormalMaterial, MeshPhongMaterial, MeshPhysicalMaterial, MeshStandardMaterial, MeshToonMaterial, MinEquation, MirroredRepeatWrapping, MixOperation, MultiMaterial, MultiplyBlending, MultiplyOperation, NearestFilter, NearestMipMapLinearFilter, NearestMipMapNearestFilter, NearestMipmapLinearFilter, NearestMipmapNearestFilter, NeverDepth, NeverStencilFunc, NoBlending, NoColors, NoToneMapping, NormalAnimationBlendMode, NormalBlending, NotEqualDepth, NotEqualStencilFunc, NumberKeyframeTrack, Object3D, ObjectLoader, ObjectSpaceNormalMap, OctahedronBufferGeometry, OctahedronGeometry, OneFactor, OneMinusDstAlphaFactor, OneMinusDstColorFactor, OneMinusSrcAlphaFactor, OneMinusSrcColorFactor, OrthographicCamera, PCFShadowMap, PCFSoftShadowMap, PMREMGenerator, ParametricBufferGeometry, ParametricGeometry, Particle, ParticleBasicMaterial, ParticleSystem, ParticleSystemMaterial, Path, PerspectiveCamera, Plane, PlaneBufferGeometry, PlaneGeometry, PlaneHelper, PointCloud, PointCloudMaterial, PointLight, PointLightHelper, Points, PointsMaterial, PolarGridHelper, PolyhedronBufferGeometry, PolyhedronGeometry, PositionalAudio, PropertyBinding, PropertyMixer, QuadraticBezierCurve, QuadraticBezierCurve3, Quaternion, QuaternionKeyframeTrack, QuaternionLinearInterpolant, REVISION, RGBADepthPacking, RGBAFormat, RGBAIntegerFormat, RGBA_ASTC_10x10_Format, RGBA_ASTC_10x5_Format, RGBA_ASTC_10x6_Format, RGBA_ASTC_10x8_Format, RGBA_ASTC_12x10_Format, RGBA_ASTC_12x12_Format, RGBA_ASTC_4x4_Format, RGBA_ASTC_5x4_Format, RGBA_ASTC_5x5_Format, RGBA_ASTC_6x5_Format, RGBA_ASTC_6x6_Format, RGBA_ASTC_8x5_Format, RGBA_ASTC_8x6_Format, RGBA_ASTC_8x8_Format, RGBA_BPTC_Format, RGBA_ETC2_EAC_Format, RGBA_PVRTC_2BPPV1_Format, RGBA_PVRTC_4BPPV1_Format, RGBA_S3TC_DXT1_Format, RGBA_S3TC_DXT3_Format, RGBA_S3TC_DXT5_Format, RGBDEncoding, RGBEEncoding, RGBEFormat, RGBFormat, RGBIntegerFormat, RGBM16Encoding, RGBM7Encoding, RGB_ETC1_Format, RGB_ETC2_Format, RGB_PVRTC_2BPPV1_Format, RGB_PVRTC_4BPPV1_Format, RGB_S3TC_DXT1_Format, RGFormat, RGIntegerFormat, RawShaderMaterial, Ray, Raycaster, RectAreaLight, RedFormat, RedIntegerFormat, ReinhardToneMapping, RepeatWrapping, ReplaceStencilOp, ReverseSubtractEquation, RingBufferGeometry, RingGeometry, SRGB8_ALPHA8_ASTC_10x10_Format, SRGB8_ALPHA8_ASTC_10x5_Format, SRGB8_ALPHA8_ASTC_10x6_Format, SRGB8_ALPHA8_ASTC_10x8_Format, SRGB8_ALPHA8_ASTC_12x10_Format, SRGB8_ALPHA8_ASTC_12x12_Format, SRGB8_ALPHA8_ASTC_4x4_Format, SRGB8_ALPHA8_ASTC_5x4_Format, SRGB8_ALPHA8_ASTC_5x5_Format, SRGB8_ALPHA8_ASTC_6x5_Format, SRGB8_ALPHA8_ASTC_6x6_Format, SRGB8_ALPHA8_ASTC_8x5_Format, SRGB8_ALPHA8_ASTC_8x6_Format, SRGB8_ALPHA8_ASTC_8x8_Format, Scene, SceneUtils, ShaderChunk, ShaderLib, ShaderMaterial, ShadowMaterial, Shape, ShapeBufferGeometry, ShapeGeometry, ShapePath, ShapeUtils, ShortType, Skeleton, SkeletonHelper, SkinnedMesh, SmoothShading, Sphere, SphereBufferGeometry, SphereGeometry, Spherical, SphericalHarmonics3, Spline, SplineCurve, SplineCurve3, SpotLight, SpotLightHelper, Sprite, SpriteMaterial, SrcAlphaFactor, SrcAlphaSaturateFactor, SrcColorFactor, StaticCopyUsage, StaticDrawUsage, StaticReadUsage, StereoCamera, StreamCopyUsage, StreamDrawUsage, StreamReadUsage, StringKeyframeTrack, SubtractEquation, SubtractiveBlending, TOUCH, TangentSpaceNormalMap, TetrahedronBufferGeometry, TetrahedronGeometry, TextBufferGeometry, TextGeometry, Texture, TextureLoader, TorusBufferGeometry, TorusGeometry, TorusKnotBufferGeometry, TorusKnotGeometry, Triangle, TriangleFanDrawMode, TriangleStripDrawMode, TrianglesDrawMode, TubeBufferGeometry, TubeGeometry, UVMapping, Uint16Attribute, Uint16BufferAttribute, Uint32Attribute, Uint32BufferAttribute, Uint8Attribute, Uint8BufferAttribute, Uint8ClampedAttribute, Uint8ClampedBufferAttribute, Uniform, UniformsLib, UniformsUtils, UnsignedByteType, UnsignedInt248Type, UnsignedIntType, UnsignedShort4444Type, UnsignedShort5551Type, UnsignedShort565Type, UnsignedShortType, VSMShadowMap, Vector2, Vector3, Vector4, VectorKeyframeTrack, Vertex, VertexColors, VideoTexture, WebGL1Renderer, WebGLCubeRenderTarget, WebGLMultisampleRenderTarget, WebGLRenderTarget, WebGLRenderTargetCube, WebGLRenderer, WebGLUtils, WireframeGeometry, WireframeHelper, WrapAroundEnding, XHRLoader, ZeroCurvatureEnding, ZeroFactor, ZeroSlopeEnding, ZeroStencilOp, sRGBEncoding, computeBox, computeSphere, pages, uvs, positions, attr, index, setTimeout, setInterval, clearTimeout, clearInterval, enroll, unenroll, active, setImmediate, clearImmediate, lines, Component, FONTS, inferResponseType, getFileNameFromURL, knownTags, isNode, wrapMethods, registrationOrderWarnings, geometryNames, Geometry, propertyTypes, registerPropertyType, isValidDefaultValue, isValidDefaultCoordinate, setupCanvas, setup, remove, MOBILE_HEAD_TAGS, inject, isSingleProperty, process, processPropertyDefinition, parseProperties, parseProperty, stringifyProperties, stringifyProperty, shaderNames, Shader, System, definePrimitive, regex, parse, stringify, isCoordinates, isCoordinate, toVector3, isWebXRAvailable, getVRDisplay, checkHeadsetConnected, checkARSupport, isTablet, isMobileDeviceRequestingDesktopSite, isOculusBrowser, isFirefoxReality, isMobileVR, isR7, isLandscape, isBrowserEnvironment, isNodeEnvironment, getComponentPropertyPath, propertyPathCache, getComponentProperty, setComponentProperty, bind, coordinates, debug, device, entity, forceCanvasResizeSafariMobile, isIE11, material, objectPool, split, styleParser, trackedControls, isGearVR, isOculusGo, throttle, throttleTick, debounce, extend, extendDeep, clone, deepEqual, diff, shouldCaptureKeyEvent, splitString, getElData, getUrlParameter, isIframed, findAllScenes, srcLoader, updateMapMaterialFromData, updateMap, updateDistortionMap, handleTextureEvents, isHLS, createPool, clearObject, removeUnusedKeys, toCamelCase, transformKeysToCamelCase, checkControllerPresentAndSetup, isControllerPresentWebVR, isControllerPresentWebXR, findMatchingControllerWebVR, findMatchingControllerWebXR, emitIfAxesChanged, onButtonEvent

three-tds-loader (unpkg)

was missing the following properties:

  1. trim
  2. BREAK
  3. RETURN
  4. f

three (unpkg)

was missing the following properties:

  1. BoxBufferGeometry
  2. CapsuleBufferGeometry
  3. CircleBufferGeometry
  4. ConeBufferGeometry
  5. CylinderBufferGeometry
as well as these 19 other properties...

DataUtils, DodecahedronBufferGeometry, ExtrudeBufferGeometry, IcosahedronBufferGeometry, ImmediateRenderObject, LatheBufferGeometry, LinearSRGBColorSpace, NoColorSpace, OctahedronBufferGeometry, PlaneBufferGeometry, PolyhedronBufferGeometry, RingBufferGeometry, SRGBColorSpace, ShapeBufferGeometry, SphereBufferGeometry, TetrahedronBufferGeometry, TorusBufferGeometry, TorusKnotBufferGeometry, TubeBufferGeometry

Generated by 🚫 dangerJS against c2d52cf

@sandersn
Copy link
Contributor Author

sandersn commented Jan 23, 2023

@Methuselah96 three should probably depend on the DOM types for WebXR instead of the shim types on DT. I doubt that three is so closely coupled to webxr that changing a few types would require any change to it at all. Let me see what I can do.

@sandersn
Copy link
Contributor Author

sandersn commented Jan 23, 2023

@Methuselah96 it looks like @types/webxr provides some things that aren't in the DOM yet. So we'll need 4.9/5.0 copies of the types.

Edit: I'm only going to do that for packages with changes in the last 18-24 months.

@Methuselah96
Copy link
Contributor

Interesting, I didn't know the DOM types included any WebXR types. Is there a special way to access those?

@Methuselah96
Copy link
Contributor

Methuselah96 commented Jan 23, 2023

Hopefully we would just need to split the types for the WebXR types package, right? (I've never split types before, but that's how I assumed it worked.)

Alternatively we can remove the OVR_multiview2 type from @types/webxr and bump the minimum required version to 4.4 (when OVR_multiview2 was added to the DOM types) to avoid having to split the types. We're only a few months away from DT dropping 4.3 support anyway.

@sandersn
Copy link
Contributor Author

re DOM including webxr types: they're available as globals. microsoft/TypeScript#52328 is the most recent update to them.

@Methuselah96
Copy link
Contributor

Methuselah96 commented Jan 23, 2023

Yeah, I was looking for anything named XR... (which are what most of the WebXR types are named) and couldn't find anything.

@sandersn
Copy link
Contributor Author

re removing OVR_multiview2 instead of updating it: much better idea. I'll do that.

then bump required TS version to 4.4 not 5.0
): WebGLRenderbuffer;
}

declare abstract class OVR_multiview2 implements OVR_multiview2 {}
Copy link
Contributor

@Methuselah96 Methuselah96 Jan 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might as well drop this as well since this is also provided by the DOM types:

Suggested change
declare abstract class OVR_multiview2 implements OVR_multiview2 {}

Copy link
Contributor

@Methuselah96 Methuselah96 Jan 23, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Or not, not sure why I thought that. Still not sure what it would be used for, but it's not included in the DOM types.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, it lets you write class extends OVR_multiview2 { } whereas the DOM types don't. I don't know if that's correct but if the types aren't broken I'm going to let an expert fix it instead of me.

@typescript-bot typescript-bot added the Revision needed This PR needs code changes before it can be merged. label Jan 23, 2023
@typescript-bot
Copy link
Contributor

@sandersn One or more reviewers has requested changes. Please address their comments. I'll be back once they sign off or you've pushed new commits. Thank you!

@typescript-bot typescript-bot added Owner Approved A listed owner of this package signed off on the pull request. and removed Revision needed This PR needs code changes before it can be merged. labels Jan 23, 2023
@sandersn sandersn merged commit b2505fd into master Jan 23, 2023
@sandersn sandersn deleted the fix-webxr-for-TS5.0 branch January 23, 2023 18:03
lgarron added a commit to cubing/cubing.js that referenced this pull request Mar 21, 2023
We need to do this manually in the lockfile until DefinitelyTyped/DefinitelyTyped#64012 has been published for `@types/webxr`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Edits multiple packages Owner Approved A listed owner of this package signed off on the pull request. Popular package This PR affects a popular package (as counted by NPM download counts). Untested Change This PR does not touch tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants